FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: Add Auth Context enabled Firestore triggers by exaby73 · Pull Request #184 · firebase/firebase-functions-python · GitHub

feat: Add Auth Context enabled Firestore triggers - #184

Merged
blidd-google merged 15 commits into
mainfrom
feat/auth-context
Apr 8, 2024
Merged

feat: Add Auth Context enabled Firestore triggers#184
blidd-google merged 15 commits into
mainfrom
feat/auth-context

Conversation

exaby73 commented Mar 18, 2024

Copy link
Copy Markdown
Contributor

No description provided.

exaby73 marked this pull request as ready for review March 19, 2024 23:15
Comment thread src/firebase_functions/firestore_fn.py Outdated

pr-Mais left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM, we just need some unit tests if feasible.

exaby73 force-pushed the feat/auth-context branch from 1b1427e to 3799007 Compare March 29, 2024 06:43
Comment thread src/firebase_functions/firestore_fn.py Outdated
blidd-google self-requested a review April 5, 2024 03:20

blidd-google left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM except nit re: naming

exaby73 requested review from blidd-google and pr-Mais April 5, 2024 18:30
blidd-google merged commit db7ad30 into main Apr 8, 2024
exaby73 deleted the feat/auth-context branch April 9, 2024 03:51
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL