add feature to enable email to admins to whitelist helpful links #728
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
carddev81
requested changes
Feb 11, 2025
PThorpe92
force-pushed
the
email_admin
branch
from
23b94a2 to
41020c8
Compare
PThorpe92
force-pushed
the
email_admin
branch
from
41020c8 to
00e6c49
Compare
jtucholski
reviewed
Feb 14, 2025
jtucholski
reviewed
Feb 14, 2025
|
|
||
| func newAllowlistRequest(user *models.User, link *models.HelpfulLink) *emailReq { | ||
| reqTime := time.Now().Format("2006-01-02 15:04:05") | ||
| emailStr := `A new Helpful Link has been added to the UnlockEd system by an Administrator. |
There was a problem hiding this comment.
@PThorpe92 Should the message in the emailStr match the html version?
I modified the message for grammar. Can you have it say?
An Administrator has added a new Helpful Link to the UnlockEd system and requests your review to confirm it is allow-listed for all users.
jtucholski
reviewed
Feb 14, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
with the following tooltip:
This Pr adds the ability for an admin to request that an email be sent to the network admin
It also adds the
emailform back to the New User form, but only when its an admin user being created. this is so we are able to use their email for reference in the request sent to the domain admin for whitelisting.