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

Add ability to add custom welcome template #169

Merged
merged 3 commits into from
Apr 21, 2024

Conversation

rrooij
Copy link
Contributor

@rrooij rrooij commented Apr 14, 2024

This adds the ability to add a custom welcome template. It works by simply adding a custom folder in the templates directory and putting a email/html/welcome.html.twig and email/text/welcome.txt.twig there (as you can read in the README).

This adds the ability to add a custom welcome template.
It works by simply adding a custom folder in the templates
directory and putting a email/html/welcome.html.twig and
email/text/welcome.txt.twig there (as you can read in the
README).
@Nowa-Ammerlaan
Copy link
Member

Willen we dit niet simpelweg voor alle templates?

@Gwildor
Copy link
Collaborator

Gwildor commented Apr 15, 2024

Willen we dit niet simpelweg voor alle templates?

@AndrewAmmerlaan eens, maar weet je welke we nog meer hebben?

@Gwildor
Copy link
Collaborator

Gwildor commented Apr 15, 2024

Als ik snel kijk hebben we in ieder geval:

  • Inschrijven nieuw lid naar het lid zelf (die deze PR oplost)
  • Inschrijven nieuw lid naar de contactpersoon/afdelingsvoorzitter (die is vrij generiek dus hoeft wellicht niet aangepast te worden)
  • Wachtwoord reset (lijkt me ook vrij generiek)
  • Inschrijven steunlid, zowel Engels als Nederlands? Geen idee hoe dat werkt. Die moet denk ik ook wel overschrijfbaar zijn.

@Nowa-Ammerlaan
Copy link
Member

  • Inschrijven steunlid, zowel Engels als Nederlands? Geen idee hoe dat werkt. Die moet denk ik ook wel overschrijfbaar zijn.

Yeah vooral deze doelde ik op. Maar het lijkt me in het kader van future-proofing ook een goed idee om dit gewoon te implementeren voor arbitraire templates (i.e. for each template; if template with same name exist in folder custom, use this instead of default template in folder templates).

@Xesau
Copy link
Contributor

Xesau commented Apr 21, 2024

Ik zou het pad van 't custom template in .gitignore zetten, verder prima

@rrooij
Copy link
Contributor Author

rrooij commented Apr 21, 2024

@Xesau Done! 👍

@Xesau Xesau merged commit a2522d9 into main Apr 21, 2024
1 check passed
@rrooij rrooij deleted the feature/customizable-welcome-email branch April 21, 2024 20:24
@Gwildor
Copy link
Collaborator

Gwildor commented Apr 22, 2024

De Socialisten heeft momenteel geen steunleden trouwens, dus dat heeft voor nu ook geen haast. Wel fijn als het binnen afzienbare tijd gebeurt, maar daar zal ik een aparte issue voor maken.

Nowa-Ammerlaan added a commit that referenced this pull request May 19, 2024
Mirrors the changes from #169

Closes: #170
Signed-off-by: Andrew Ammerlaan <andrewammerlaan@gentoo.org>
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.

4 participants