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

Admin rejecting account request: give actual account in the email template #13102

Closed
damithc opened this issue Apr 30, 2024 · 0 comments · Fixed by #13118
Closed

Admin rejecting account request: give actual account in the email template #13102

damithc opened this issue Apr 30, 2024 · 0 comments · Fixed by #13118
Labels
enhancement New feature or request p.Low Very little impact; unlikely to do in the near future

Comments

@damithc
Copy link
Contributor

damithc commented Apr 30, 2024

When the admin is rejecting an account request using 'reject with reason' option, one of the possible reasons is that there is already an account for that email. In that case, the user is suggested to login using the existing account.

image

image

Current: the suggested login account is the same as the email address specified in the account request
Suggested:

  1. Specify the Google account of the user instead, as that's what the user needs to use to login

  2. Add the following info to the instructions.

    If you are logged into multiple Google accounts, remember to logout from other Google accounts first, or use an incognito Browser window. Let us know (with a screenshot) if that doesn't work.
@damithc damithc added p.Low Very little impact; unlikely to do in the near future enhancement New feature or request labels Apr 30, 2024
@Andy-W-Developer Andy-W-Developer mentioned this issue May 21, 2024
Merged
weiquu added a commit that referenced this issue Jun 28, 2024
@Andy-W-Developer @domoberzin @weiquu
[#13102] Admin rejecting account request: give actual account in the …
efeffd4 
…email template (#13118)

* Change existingEmail to googleId

* Add extra info to instructions

* Fix missing googleId if multiple accounts exist

* Add function name to test name

* Move account search to function

* Change missing googleId string to be shorter

* Add check for accounts with no googleId

* Add tests for replaceGoogleId

* Change instructor search result to use createBuilder

* Change a test name to fit convention

---------

Co-authored-by: domoberzin <[email protected]>
Co-authored-by: Wei Qing <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request p.Low Very little impact; unlikely to do in the near future
Projects
None yet
Milestone
No milestone
1 participant
@damithc