Skip to content

Conversation

@Tiedye
Copy link

@Tiedye Tiedye commented Feb 3, 2019

Checklist

Description

The method AngularFirestoreCollection.add is not consistent with AngularFirestoreCollection.doc and returns a class of type Promise<firebase.firestore.DocumentReference> instead of a Promise<AngularFirestoreDocument<T>>.

This is a breaking change of a previously undocumented behaviour.

Code sample

This

const newDoc = this.afStore.doc<A>(await this.afStore.collection<A>('drafts').add({...}));

becomes

const newDoc = await this.afStore.collection<A>('drafts').add({...});

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

1 similar comment
@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

@Tiedye
Copy link
Author

Tiedye commented Feb 3, 2019

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

1 similar comment
@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes and removed cla: no labels Feb 3, 2019
@jamesdaniels jamesdaniels requested a review from davideast May 31, 2019 13:28
@jamesdaniels
Copy link
Member

FYI I've asked David to review as this is an API change. As such we wouldn't be able to merge to v6 if we want to go this direction.

@jamesdaniels jamesdaniels added this to the 6.0.0 milestone May 31, 2019
@jamesdaniels jamesdaniels modified the milestones: 6.0.0, 7.0.0 Nov 11, 2020
@jamesdaniels
Copy link
Member

Thanks for this contribution, we opted to go a different direction in v7 however and no longer have a AngularFirestoreDocument (outside of compat which I consider for the most-part feature complete).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants