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

Invite to organization has hard-coded email text #1527

Closed
rossjones opened this issue Feb 19, 2014 · 3 comments
Closed

Invite to organization has hard-coded email text #1527

rossjones opened this issue Feb 19, 2014 · 3 comments

Comments

@rossjones
Copy link
Collaborator

@rossjones rossjones commented Feb 19, 2014

Would be useful if the email text in get_invite_body() (from mailer.py) loaded a template with the text so that extensions can override it. When working around #1506 it is likely that the email template will need to be changed, and/or other users might want to change the text of the email.

@nigelbabu
Copy link
Contributor

@nigelbabu nigelbabu commented Feb 21, 2014

Hi @rossjones, at the dev meeting, we felt this was a good idea. The thing to consider is translations and having the subject changeable as well.

@amercader amercader added this to the CKAN 2.4 milestone Oct 22, 2014
amercader added a commit that referenced this issue Jun 12, 2015
Read the reset password and invite user emails body from text templates
(which are magically translated!) and the subjects from config options.
amercader added a commit that referenced this issue Jun 12, 2015
amercader added a commit that referenced this issue Jun 12, 2015
amercader added a commit that referenced this issue Jun 12, 2015
amercader added a commit that referenced this issue Jun 15, 2015
The current invite email doesn't really give you much information about
what you are actually signing for. This adds the organization title and
the role that you've been assigned to the invite email.
amercader added a commit that referenced this issue Jun 15, 2015
As per @wardi's suggestion. Added a new example plugin with tests
wardi added a commit that referenced this issue Jan 7, 2016
[#1527] Allow to customize emails sent from CKAN
@hbunke
Copy link

@hbunke hbunke commented Jan 27, 2016

What's the status of this issue? The code has been in master for some time now (6654999), but nothing seems to have made it into a release so far.

@TkTech TkTech closed this May 10, 2016
@davidread
Copy link
Contributor

@davidread davidread commented May 24, 2016

This issue was closed due to inactivity. Feel free to reopen if you have more feedback or are interested it working on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
6 participants