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

Fix/submission emails #2405

Conversation

clari182
Copy link
Collaborator

@clari182 clari182 commented Nov 10, 2023

Emails are being sent to users that have submitted submissions in the past, even for new submissions.
They should only receive notifications for the ones they have created.

Test URL: https://deploy-preview-30--clarayoudale-staging.netlify.app/
test users:
[email protected]
[email protected]
[email protected]
(If you want you can create your own test user(s))

How to test:

  • Login with one of the users
  • Submit a new report (or multiple reports if you want)
  • Login with another user
  • Submit a new report (or multiple reports if you want)
  • On the submission page https://deploy-preview-30--clarayoudale-staging.netlify.app/apps/submitted/ review and add all your new submissions as incidents (for this you need to be logged in with [email protected])
  • Trigger a postman request to https://realm.mongodb.com/api/client/v2.0/app/aiidstitch2-qoueg/graphql
    with body
mutation ProcessNotifications {
      processNotifications
    }

(ask Clara for request's headers)

  • Users should only receive email for the submission they've created and not both.

@clari182 clari182 marked this pull request as draft November 10, 2023 17:52
Copy link

netlify bot commented Nov 10, 2023

Deploy Preview for staging-aiid canceled.

Name Link
🔨 Latest commit caaebb8
🔍 Latest deploy log https://app.netlify.com/sites/staging-aiid/deploys/654e6de23cc94b000818f6c7

@clari182 clari182 marked this pull request as ready for review November 13, 2023 12:23
@clari182 clari182 requested a review from cesarvarela November 13, 2023 12:23
@cesarvarela cesarvarela merged commit a2931a5 into responsible-ai-collaborative:staging Nov 13, 2023
3 checks passed
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.

2 participants