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.

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

prepping for new care team alerting invites #125

Merged
merged 2 commits into from
Sep 1, 2023
Merged

Conversation

ewollesen
Copy link
Contributor

Builds on top of #122 , so if that hasn't merged yet, it would probably be easier to review that one first. :)

tjotala
tjotala previously approved these changes Sep 1, 2023
Copy link
Member

@tjotala tjotala left a comment

Choose a reason for hiding this comment

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

LGTM

No change to existing tests using it, but I'll need to be able to mock
multiple users to test the new invites system.

Part of BACK-2500
@ewollesen
Copy link
Contributor Author

Updated to sit atop #122, no changes aside from rebase (and no rebase conflicts)

Copy link
Member

@tjotala tjotala left a comment

Choose a reason for hiding this comment

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

LGTM v2

@ewollesen ewollesen merged commit b1e0d1d into master Sep 1, 2023
3 checks passed
@ewollesen ewollesen deleted the eric/send-invite-v2.1 branch September 1, 2023 20:59
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