-
Notifications
You must be signed in to change notification settings - Fork 89
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
notifications: add submission action notifications #1447
notifications: add submission action notifications #1447
Conversation
Specific accept action notification for the community submission. Other actions will follow a similar pattern. |
745a1ad
to
aa47b12
Compare
aa47b12
to
bddb076
Compare
if we merge this PR we will send 2 emails for access request accept action, see: |
bddb076
to
2c0199f
Compare
These notifications are specific to the community submission/inclusion actions. Do access requests and inclusion requests share the same service? If not, we should be good here. |
OK for separate PR! |
Just tested locally and it did not send two mails/notifications on access request accept action. |
actions: send notification on submission accept action
2c0199f
to
cbede2c
Compare
actions: send notification on submission accept action
❤️ Thank you for your contribution!
Description
closes inveniosoftware/invenio-requests#339
Send notification when executing a submission request action.
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:
Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that: