Skip to content

Use Navigation in Firestore #1268

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
merged 9 commits into from
Mar 15, 2021
Merged

Use Navigation in Firestore #1268

merged 9 commits into from
Mar 15, 2021

Conversation

samtstern
Copy link
Contributor

See #1217

Change-Id: I6aa24b3242690820ae15e8fc5042adb6de60dee8
Change-Id: Iabde582e48e5b0998870e96a5065004ba2006c2e
@samtstern samtstern requested a review from thatfiredev March 15, 2021 15:12
Change-Id: I169cb0b1a227590255909b303a9394aea1ab7a6d
Copy link
Member

@thatfiredev thatfiredev left a comment

Choose a reason for hiding this comment

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

@samtstern 98% looks just, we just need to fix a couple of things and I have a few suggestions:

samtstern and others added 3 commits March 15, 2021 11:52
…ts/java/MainActivity.java

Co-authored-by: Rosário Pereira Fernandes <[email protected]>
Change-Id: I82acd91dec0dc6119150dca7ef1888676de6587d
Copy link
Member

@thatfiredev thatfiredev left a comment

Choose a reason for hiding this comment

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

Changes look good! 🎉

@thatfiredev thatfiredev mentioned this pull request Mar 15, 2021
16 tasks
Change-Id: I11124189fc3b03e78d91dbb031bfc14e630e070d
Change-Id: I484f2bf80af2cfc38c96518a7755461a9ffb5af9
Change-Id: Ia30bb29037720fe5252f8a07756e8810030435a8
@samtstern samtstern merged commit 12d0ace into master Mar 15, 2021
@thatfiredev thatfiredev deleted the ss-firestore-navigation branch May 3, 2024 00:23
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.

2 participants