-
Notifications
You must be signed in to change notification settings - Fork 112
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add Document.path property with Tests. Closes #99
Update Tests to sync with Shields and Github caching. Fix Tests that couldn't fail. Update README and packages.
- Loading branch information
1 parent
773c237
commit c8641b1
Showing
6 changed files
with
200 additions
and
90 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
c8641b1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm getting this error even after updating everything to V8
TypeError: firestore.query(...).where is not a function
c8641b1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have tried Capitalize word as well. This issue occur after 26th version
TypeError: firestore.query(...).Where(...).execute is not a function
c8641b1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you try
firestore.query(...).Where(...).Execute()
with capital "Execute"?c8641b1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you and it worked !
One more thing its not related to this issue,
How do I update more than one document once. Lets say, I have to update 10 documents. It is not good to loop the given updated query 10 times.
This is a most common use case and firestore has that capability. Can I do that using this library ?
One more suggestion. It would be nice, if there is any place to add public comments 👍
c8641b1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your last comment specifies "Execute" that is in lowercase. Since Javascript is case-sensitive, I take that into consideration. Since you commented here, you should see lots of Tests that all work. One example of a working test uses your same query.
You need to use a loop at the moment. Batched Writes (#65) is still an open issue.
You should be opening an issue for a place to talk about any issue/comment/suggestion/bug. 💯
c8641b1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your quick response.
We are planing to use this library and hope that batch methods will updated soon.
I would like to talk with your personally if you don't mind. Can you provide me any contact ?
c8641b1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's great news! However, I can't currently dedicate the time to add that feature in myself. As this is a community-driven project, anyone following the guidelines can contribute. And any questions you have can be addressed to the community as a whole (thousands of active users).