impact-graph icon indicating copy to clipboard operation
impact-graph copied to clipboard
verified publisher icon Giveth

send email when project receives or loses vouched badge

Open divine-comedian opened this issue 1 year ago • 5 comments

We should handle informing the user when they receive the vouched badge as well as when they have it taken away. This should be done via an email sent through Ortto.

Currently we use an activity called project-verification that sends a verified-status variable - depending on the value of this variable we send a different email to the user. I've added two new conditions and emails that we can use for this purpose

  • vouched - triggers email for when a project becomes vouched
  • unvouched - triggers email for when a project become vouched

Note: If a project becomes verified and givbacks eligible in a single action we should ONLY send the GIVbacks eligible email conversely, if a project loses it's badge either from rejection of its applicaiton or revoking it's givbacks eligible badge we should only send those email, not the "unvouched" email

steps needed to accomplish

  • [x] - setup activity in ortto
  • [x] - draft email for when project receives vouched badge in Ortto
  • [x] - draft email for when project has lost vouched badge in Ortto
  • [ ] - setup activity trigger in BE when admin changes verified status of project

AC

  • [ ] Admin can change vouched/verified status of project from admin panel
  • [ ] when project receives vouched badge email is sent to project owner
  • [ ] when project has vouched badge removed an email is sent to project owner
  • [ ] when project has no vouched badge and receives both badges at once (givbacks & vouched badge) ONLY givbacks email is sent
  • [ ] when project has both badges and loses both in a single action, ONLY givbacks email is sent.

divine-comedian avatar Oct 08 '24 15:10 divine-comedian

@MohammadPCh what is the update on this issue?

divine-comedian avatar Nov 04 '24 06:11 divine-comedian

@divine_comedian I've submitted a PR for it, and it's awaiting review. Apologies for not mentioning this issue earlier—I'll add it now!

MohammadPCh avatar Nov 04 '24 07:11 MohammadPCh

@kkatusic would you be able to pick this one up too when you finish the issue related to email verification?

divine-comedian avatar Nov 19 '24 15:11 divine-comedian

@kkatusic would you be able to pick this one up too when you finish the issue related to email verification?

I will, thx ;)

kkatusic avatar Nov 20 '24 12:11 kkatusic

@divine-comedian, @MohammadPCh pushed changes to the BE, did someone test it?

kkatusic avatar Jan 28 '25 10:01 kkatusic