Skip to content

Move DocumentId to public space #571

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

Merged
4 commits merged into from
Jun 26, 2019
Merged

Move DocumentId to public space #571

4 commits merged into from
Jun 26, 2019

Conversation

ghost
Copy link

@ghost ghost commented Jun 26, 2019

No description provided.

@googlebot googlebot added the cla: yes Override cla label Jun 26, 2019
@ghost
Copy link
Author

ghost commented Jun 26, 2019

/test device-check-changed

@ghost ghost force-pushed the wuandy/DocumentIdGoesPublic branch from 8c0e7da to ad38eb4 Compare June 26, 2019 17:44
@ghost ghost requested a review from mikelehen June 26, 2019 18:01
@ghost ghost assigned mikelehen Jun 26, 2019
Copy link
Contributor

@mikelehen mikelehen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with nits. I made some suggestions to the javadoc, but I know this got some scrutiny in the API review already. If you feel like my feedback is contrary to feedback you got previously, feel free to punt.

@mikelehen mikelehen assigned ghost and unassigned mikelehen Jun 26, 2019
Copy link
Author

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed your comments, PTAL.

Copy link
Contributor

@mikelehen mikelehen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suuper close, but I just remembered that we should add a firebase-firestore/CHANGELOG.md entry for this now that it's going to be public (which will then get pulled into https://firebase.google.com/support/release-notes/android). Sorry!

@ghost
Copy link
Author

ghost commented Jun 26, 2019

CHANGELOG added.

@ghost ghost assigned mikelehen and unassigned ghost Jun 26, 2019
Copy link
Contributor

@mikelehen mikelehen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@mikelehen mikelehen assigned ghost and unassigned mikelehen Jun 26, 2019
@ghost ghost merged commit a478b26 into master Jun 26, 2019
schmidt-sebastian pushed a commit that referenced this pull request Jun 27, 2019
* Move DocumentId to public space

* add nest object testing

* add changelog
@rlazo rlazo deleted the wuandy/DocumentIdGoesPublic branch September 27, 2019 14:53
@firebase firebase locked and limited conversation to collaborators Oct 9, 2019
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Override cla size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants