Skip to content

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