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

submission notification is not working #8

Closed
Tracked by #151 ...
syphax-bouazzouni opened this issue Jan 17, 2022 · 2 comments
Closed
Tracked by #151 ...

submission notification is not working #8

syphax-bouazzouni opened this issue Jan 17, 2022 · 2 comments
Labels

Comments

@syphax-bouazzouni
Copy link

syphax-bouazzouni commented Jan 17, 2022

An email notification normally would be sent if any submission is parsed. But it seems that we have never seen it worked.

The code responsible on sending it is here : https://github.com/ontoportal-lirmm/ontologies_linked_data/blob/master/lib/ontologies_linked_data/models/ontology_submission.rb#L1923

And it's an "ensure" clause, which means that whatever happen (exception or not ) the email notification will be sent.

The mail is defined here : https://github.com/ontoportal-lirmm/ontologies_linked_data/blob/master/lib/ontologies_linked_data/utils/notifications.rb#L63

@syphax-bouazzouni
Copy link
Author

syphax-bouazzouni commented Mar 31, 2022

Issue updates

It seems that the submission notification is sent for only the users that are subscribed to the ontology, with a subscription of type "PROCESSING" or "ALL" (and not of type "NOTE")

Solution

Change the mail recipient's behavior for the notifications :

  • new_note => notify subscribed to the ontology(separately) (note: ontology administrators are not subscribed by default?)
  • submission_processed => notify subscribed(separately) + ontology_adminstrators and support mails (all grouped)(changed: added support and ontology_adminstrators mails)
  • remote_ontology_pull => notify ontology_adminstrators and support mails (all grouped)
  • new_user => notify support mails (grouped)
  • new_ontology => notify support mails (grouped)
  • reset_password => notify the user that is resetting it password
  • obofoundry_sync => notify the support mails (grouped)

working branch: https://github.com/ontoportal-lirmm/ontologies_linked_data/compare/master...ontoportal-lirmm:pl/refactor-notifications?expand=1

Tests

https://github.com/ontoportal-lirmm/ontologies_linked_data/actions/runs/2077179672

Deploy on

  • testportal
  • stageportal
  • agrportal
  • bioportal

@syphax-bouazzouni
Copy link
Author

fixed :
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant