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

fix: more information in invitation email templates #577

Merged
merged 2 commits into from
Oct 12, 2022

Conversation

ewan-escience
Copy link
Collaborator

More information in invite email templates

Changes proposed in this pull request:

  • Add the name of the software/project/organisation in the invite email template

How to test:

  • docker-compose build frontend && docker-compose up --scale scrapers=0
  • Login as an admin
  • Create invites for software, projects and invitations
  • Check that the email templates contain the names of the unit

Closes #451

PR Checklist:

  • Increase version numbers in docker-compose.yml
  • Link to a GitHub issue
  • Update documentation
  • Tests

Copy link
Contributor

@dmijatovic dmijatovic left a comment

Choose a reason for hiding this comment

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

Nice work!
I do have two remarks.

I suggest to change the email subjects also into:

  • Maintainer invite for RSD organisation {organisation_name}
  • Maintainer invite for RSD project {project_title}
  • Maintainer invite for RSD software {software_name}

The line breaks (\n) are not properly applied by my email client (Outlook online). But this problem was present from the beginning, so not sure if we can improve it.

@dmijatovic dmijatovic self-requested a review October 12, 2022 09:25
Copy link
Contributor

@dmijatovic dmijatovic left a comment

Choose a reason for hiding this comment

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

Nice addition! Thanks.

@ewan-escience ewan-escience merged commit 92a6bd4 into main Oct 12, 2022
@ewan-escience ewan-escience deleted the 451-email-templates branch October 17, 2022 17:48
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.

More information required in maintainer invitation mail templates
2 participants