Skip to content

Support Email Config Option#141

Merged
JVickery-TBS merged 4 commits intomasterfrom
feature/support-email-config
May 27, 2025
Merged

Support Email Config Option#141
JVickery-TBS merged 4 commits intomasterfrom
feature/support-email-config

Conversation

@JVickery-TBS
Copy link
Contributor

feat(dev): support email config;

  • Configurized the support email.

- Configurized the support email.
- Added change log file.
@wardi
Copy link
Member

wardi commented May 23, 2025

technically this makes ckanext-recombinant dependent on ckanext-canada, which already depends on recombinant so that's not great.

Let's create the support_email_address helper here in recombinant (and use it in the views instead of reading the config) then in ckanext-canada we can override the email with the one from our configuration by overriding the support_email_address helper to remove the dependency loop

- Remove dependency on Canada plugin.
- EOF new line.
@JVickery-TBS JVickery-TBS merged commit c0453ac into master May 27, 2025
6 checks passed
@JVickery-TBS JVickery-TBS deleted the feature/support-email-config branch May 27, 2025 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants