-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Allow rostering directly from email #375
Allow rostering directly from email #375
Conversation
Hi @supergi0, thanks for your contribution ! |
Thanks for taking the time to contribute, @supergi0 ! 🎉 I'd really appreciate if you also put in some more context into the commit messages. The PR description details the changes made, etc., but it would be nice to have the "why" and some more context about the changes in the PR, to make the commits more understandable. I'll let @SibiAkkash and @Joe2k do the more detailed review, that they already seem to be doing. Thanks again for the contribution! |
ee00eb5
to
70999b7
Compare
654a089
to
042aa69
Compare
042aa69
to
b480f17
Compare
On Team Admin side
On Player side
To test
EMAIL_SECRET_KEY
environment variable.Email ss
https://imgur.com/a/jj5QcC5
Series invitation error response UI
Series invitation accept UI
Series invitation decline UI