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

Allow to send notifications when Admins cancel jobs #19547

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

davelopez
Copy link
Contributor

Stretch goal for #19487, thank you, @GarethPrice-Aus for proposing it.

Sending job canceled notifications

Admins can now decide to notify users when they cancel any of their jobs.

After selecting the jobs to cancel and providing a message, admins can enable the switch to send notifications. One notification will be sent per user (even if multiple jobs belonging to the same user were canceled).

image

The notification will be displayed to the user after a few seconds and potentially get an email if the user has the appropriate notification settings. If multiple jobs for the same user were canceled then all of them will be listed in the notification.

image

Clicking any of the jobs (tool IDs) listed in the notification will display a new page with the job details.

image

Small API change

Listing jobs with user_details will include the user_id in addition to the existing user_email for convenience.

How to test the changes?

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

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.

1 participant