Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

createUserWithEmailAndPassword returns object with user property #138

Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

createUserWithEmailAndPassword returns object with user property #138

wants to merge 31 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 25, 2019

According to Google, this FirebaseAuth method should return a
UserCredential instead of just a User (the User is the value of the user property of UserCredential).
https://firebase.google.com/docs/reference/node/firebase.auth.Auth#createUserWithEmailAndPassword

BrianChapman and others added 24 commits January 23, 2019 08:38
…l the onSnapshot callbacks no mater where the flush is called from.
When a new user is created in firebase, they are automatically
signed in.
Source: https://firebase.google.com/docs/auth/web/password-auth
When a new user is created in firebase, they are automatically
signed in.
Source: https://firebase.google.com/docs/auth/web/password-auth
* issue-81-onsnapshot:
  Move onSnapshot handers to the queue so that a call to flush will call the onSnapshot callbacks no mater where the flush is called from.
  onSnapshot should call callback if no data changed and includeMetadataChanges is true.
  Add onSnapshot functionality to doc, collection and query
* issue-81-onsnapshot:
  onSnapshot should call immediatly and then after data changes.
@luan
Copy link

luan commented May 25, 2019

Any news on getting this merged? This is blocking my upgrading the firebase package.

@ghost
Copy link
Author

ghost commented May 26, 2019

Any news on getting this merged? This is blocking my upgrading the firebase package.

soumak77 hasn't been active on github since last September and I'm not sure anyone else has write access. You can use my fork for now and I could merge the other PRs

CharlesThomasJohnson added 3 commits August 27, 2019 10:37
Support specifying _updateTime and _createTime when mocking firestore doc
@dmurvihill
Copy link

Hi all,

It has transpired that @soumak77 passed away last October. We are in the process of transferring ownership of the firebase-mock npm package. Are you interested in merging your fork?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants