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

Store email address in medperf database #511

Merged
merged 4 commits into from
Dec 8, 2023
Merged

Conversation

hasan7n
Copy link
Contributor

@hasan7n hasan7n commented Nov 26, 2023

In the near future we may implement sending email notifications. Access tokens should include a custom claim named https://medperf.org/email which will include the user's email address. This PR will store the user's email address in the medperf database the first time the user accesses the server

@hasan7n hasan7n requested a review from a team as a code owner November 26, 2023 22:20
@hasan7n hasan7n had a problem deploying to testing-external-code November 26, 2023 22:20 — with GitHub Actions Failure
Copy link
Contributor

github-actions bot commented Nov 26, 2023

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@hasan7n hasan7n temporarily deployed to testing-external-code November 27, 2023 09:08 — with GitHub Actions Inactive
Copy link
Contributor

@aristizabal95 aristizabal95 left a comment

Choose a reason for hiding this comment

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

Looks great!

@hasan7n hasan7n had a problem deploying to testing-external-code December 8, 2023 16:32 — with GitHub Actions Failure
@hasan7n hasan7n merged commit d0dd5a5 into mlcommons:main Dec 8, 2023
6 of 7 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants