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

[TGDK][Feature] Add Configurable Option to Enable or Disable 'Skip Email Notification' Button #4557

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

SSanchez7
Copy link

Important

This implementation is part of a set of features and fixes developed within the context of a project for the TGDK academic journal, with the goal of customizing Janeway to meet the journal's specific needs, which may also be extended to other contexts.

Problem / Objective

By default, many interactions between users on the platform (such as editor assignments, reviewer assignments, etc.) include the option to skip sending an email notification to the other party involved. This is implemented via a button on the respective notification page.

image

While this flexibility is valuable for some workflows, many journals require administrators to enforce the sending of these notifications as part of their editorial policies. The current setup does not allow journals to disable the skip option, which can lead to inconsistencies in communication practices.

Solution

This feature introduces a configurable option, "Enable Skip Send Email", to allow journals to enable or disable the "Skip Email Notification" button

image

The option is added to the Review Settings and is activated by default to align with the platform's current behavior.

Journals can disable this setting to enforce mandatory email notifications for all user interactions.
Impact on User Interface

When disabled, the "Skip Email Notification" button is no longer displayed on relevant notification pages, ensuring that all interactions automatically trigger email notifications.

@SSanchez7 SSanchez7 marked this pull request as ready for review December 21, 2024 11:02
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.

1 participant