Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[INTF23] Non Authenticated BP members #42

Merged
merged 3 commits into from
Nov 20, 2023
Merged

[INTF23] Non Authenticated BP members #42

merged 3 commits into from
Nov 20, 2023

Conversation

HeetShah
Copy link
Contributor

Notion ticket link

Ticket Name

Implementation description

Steps to test

What should reviewers focus on?

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@HeetShah HeetShah added the ready to merge PR is approved and ready to merge into `main` label Nov 17, 2023
@HeetShah HeetShah marked this pull request as ready for review November 20, 2023 04:24
@HeetShah HeetShah force-pushed the INTF23-non-auth-firebase-users branch from 308de8d to ecf6f6c Compare November 20, 2023 04:43
@HeetShah HeetShah merged commit 475bb29 into main Nov 20, 2023
1 check passed
@HeetShah HeetShah deleted the INTF23-non-auth-firebase-users branch November 20, 2023 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ready to merge PR is approved and ready to merge into `main`
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@HeetShah