Typescript Cloud function not compiling

Question

I have following cloud function to get data from the firebase datastore but it doesn't compile and am not sure how to fix this.

import * as admin from 'firebase-admin';
import * as functions from 'firebase-functions';

admin.initializeApp(functions.config().firebase);

const db = admin.firestore();
const usersObj = db.collection('users')

export const getUsers = async () => {
    let allUsers: Array<any> = [];
    await usersObj.get().then(users => {
        users.forEach(user => {
            allUsers[user.id] = user.data();
        });
    });
    return allUsers;
}

error: error TS7015: Element implicitly has an 'any' type because index expression is not of type 'number'. allOrders[order.id] = order.data();

I think that's the problem. Array index should be a number. You could change allUsers to be an object, or use forEach index as array index instead of user's GUID — Héctor Valls
– Héctor Valls, Commented May 15, 2019 at 7:33

Carsten · Accepted Answer · 2019-05-15 08:01:03Z

1

If you want to transform your array elements into a new array, you could also use Array.map.

allUsers = users.map(user => user.data());

answered May 15, 2019 at 8:01

Carsten

9536 silver badges7 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

niklodeon Over a year ago

actually what I wanted to do was something like users.forEach(user => { allUsers.push({ id: user.id, ...user.data() }); }); I don't know how to do it using .map

Carsten Over a year ago

Something like this: allUsers = users.map(user => Object.assign({ id: user.id }, user.data()));

niklodeon Over a year ago

doesn't seems to work. getting error Property 'map' does not exist on type 'QuerySnapshot'.

Héctor Valls · Accepted Answer · 2019-05-15 07:38:34Z

0

You can't index an array using a string. To solve it, you can:

Use forEach index as array index:

users.forEach((user, i) => {
    allUsers[i] = user.data();
});

or push items to the allUsers array:

users.forEach(user => {
    allUsers.push(user.data());
});

answered May 15, 2019 at 7:38

Héctor Valls

26.4k43 gold badges151 silver badges267 bronze badges

2 Comments

niklodeon Over a year ago

great. thanks... I was using push before but removed it during fixing other type issues with allUsers...

Héctor Valls Over a year ago

great, glad to help

Collectives™ on Stack Overflow

Typescript Cloud function not compiling

2 Answers 2

3 Comments

2 Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

2 Comments

Related