Skip to content

feat: update emails with branding template#234

Merged
minh-biocommons merged 1 commit intomainfrom
email-ui-update
Mar 31, 2026
Merged

feat: update emails with branding template#234
minh-biocommons merged 1 commit intomainfrom
email-ui-update

Conversation

@minh-biocommons
Copy link
Copy Markdown
Collaborator

@minh-biocommons minh-biocommons commented Mar 25, 2026

Description

AAI-786: Templates on emails send from aai-backend

Update emails with branding

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have added unit / integration tests that prove my fix is effective or that my feature works
  • I have run all tests locally and they pass
  • I have updated the documentation (if applicable)
  • For any new secrets, I have updated the shared spreadsheet and the GitHub Secrets.

Screenshots

image image image image image image

Copy link
Copy Markdown
Contributor

@amandazhuyilan amandazhuyilan left a comment

Choose a reason for hiding this comment

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

Hi @minh-biocommons do you mind putting up some screenshots for the updated emails - thanks!

@amandazhuyilan
Copy link
Copy Markdown
Contributor

Thanks @minh-biocommons - for consistency, do we want to all add the black button "Go to BioCommons Access Portal" instead of the url link to 'please visit the "BioCommons Access Portal"' in the emails (ex: notification emial for username update, Confirm updated email, notification email for email update)

Copy link
Copy Markdown
Contributor

@amandazhuyilan amandazhuyilan left a comment

Choose a reason for hiding this comment

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

Sorry for leaving this open for so long! Good to go

@minh-biocommons minh-biocommons merged commit 482c424 into main Mar 31, 2026
5 checks passed
@minh-biocommons minh-biocommons deleted the email-ui-update branch March 31, 2026 23:20
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