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

Notifications sends separate emails for archived status with vague content #190

Closed
jvendetti opened this issue Feb 14, 2024 · 3 comments
Closed
Assignees

Comments

@jvendetti
Copy link
Member

jvendetti commented Feb 14, 2024

I'm listed as an administrator for the ARO ontology. I got an email at 6:03pm that a new version was processed with a title of "Antibiotic Resistance Ontology Parsing Success". The content is:

Antibiotic Resistance Ontology (ARO) was processed for use in BioPortal. Here are the results:

Parsed successfully
Indexed terms for search
Indexed properties for search
Class metrics calculated
Created submission version diff successfully
...

I got a second email (also at 6:03pm) with the identical title of "Antibiotic Resistance Ontology Parsing Success". The content is:

Antibiotic Resistance Ontology (ARO) was processed for use in BioPortal. Here are the results:

Archived
...

Since I know BioPortal's internals, I read this and understood that since a new submission was uploaded, we archived the previous submission. However, there are lots of BioPortal users who administer ontologies and don't know anything about our internals. I find the content potentially confusing to users who might wonder what archived is even referring to with regard to their ontology. I also don't think we should generate multiple messages when new submissions are processed.

@jvendetti jvendetti changed the title Notifications is sending separate emails for archived status with vague content Notifications sends separate emails for archived status with vague content Feb 14, 2024
@alexskr
Copy link
Member

alexskr commented Feb 14, 2024

I agree, second email notification about submission archival is not needed
see also #185 (comment)

@alexskr alexskr self-assigned this Feb 14, 2024
@alexskr
Copy link
Member

alexskr commented Feb 14, 2024

relates to ncbo/bioportal-project#291

@jvendetti
Copy link
Member Author

Production was rolled back on May 8th to v5.29.1. This behavior is occurring again. Reopening this as a reminder to make sure the fixes are correctly re-deployed.

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

No branches or pull requests

2 participants