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

Initial draft of 3.4 release notebook #1063

Merged
merged 26 commits into from
Mar 8, 2023
Merged

Conversation

NateWr
Copy link
Contributor

@NateWr NateWr commented Feb 20, 2023

No description provided.

@withanage
Copy link
Member

@NateWr somethings, which comes to my mind , specially related to plugins, which may be relevant, specially for admins who have their own plugins

  • Plugins shold also have the language folders 'en' etc.
  • Email sending rewrite , if plugins send emails. e.g. is in orcidProfile Plugin. You have to reinstall the plugin to get the new keys installed into the email_templates_default_data table.
  • namespacing and removing .inc. extension also for plugins for optimizing

@NateWr
Copy link
Contributor Author

NateWr commented Feb 20, 2023

Thanks @withanage. Can you say more about this?

Email sending rewrite , if plugins send emails. e.g. is in orcidProfile Plugin. You have to reinstall the plugin to get the new keys installed into the email_templates_default_data table.

The system should port over any existing emails, but maybe something is missing here?

@withanage
Copy link
Member

withanage commented Feb 20, 2023

Thanks @withanage. Can you say more about this?

Email sending rewrite , if plugins send emails. e.g. is in orcidProfile Plugin. You have to reinstall the plugin to get the new keys installed into the email_templates_default_data table.

The system should port over any existing emails, but maybe something is missing here?

For me the name attribute was missing in the orcidProfile Plugin emailTemplates.xml , until then the emails did not work. Vitaliey helped me finding that, may be if plugin developers do have emailTemplates.xml which does not have a name attribute , it shold fail.

@jonasraoni
Copy link
Contributor

You may need to update the locale setting in the [i18n] section of config.inc.php

I think it might be worth to add a pre-flight check for this :)

@NateWr
Copy link
Contributor Author

NateWr commented Feb 20, 2023

For me the name attribute was missing

Hmm, do you remember what the error was? The upgrade migration is supposed to automatically add a name value for every template, even if one is not defined.

https://github.com/pkp/pkp-lib/blob/84c5a17c958948a4d5b29bdbd935c89ff835d17a/classes/migration/upgrade/v3_4_0/I5716_EmailTemplateAssignments.php#L252-L260

Do you know if you ran the I5716_EmailTemplateAssignments migration when you encountered the error? It should be run during the normal upgrade process...

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.

5 participants