decidim icon indicating copy to clipboard operation
decidim copied to clipboard

Add notification to valuators when a proposal is assigned

Open mllocs opened this issue 1 year ago • 5 comments

:tophat: What? Why?

Sends a notification to the valuator when a proposal is assigned to them.

:pushpin: Related Issues

Closes https://github.com/decidim/decidim/issues/12909

Testing

  • Assign a proposal to a valuator, the valuator receives a notification and an email. https://decidim-lot2.populate.tools/admin/participatory_processes/branch-highway/components/4/manage/
  • The valuator can opt out from receiving a notification when a proposal is assigned to them. https://decidim-lot2.populate.tools/notifications_settings

:camera: Screenshots

Screenshot 2024-06-19 at 09 30 15 Screenshot 2024-06-19 at 09 29 17

:hearts: Thank you!

mllocs avatar Jun 19 '24 07:06 mllocs

This PR is on hold until https://github.com/decidim/decidim/pull/12994 is merged.

mllocs avatar Jun 19 '24 07:06 mllocs

The notification being received is this:

65964

Please @NilHomedes say if it's OK or we should adjust anything.

furilo avatar Jul 01 '24 10:07 furilo

Please @NilHomedes say if it's OK or we should adjust anything.

Yes, looks good to me. Just to clarify, the proposal-name should not be a link. So, in the notification, we only show a link to the admin panel.

I'm trying to test this, but I'm not receiving the notification email, even though I'm receiving other emails like the password recovery instructions.

NilHomedes avatar Jul 01 '24 11:07 NilHomedes

How is your conf? (I had the same issue ;)

58929

furilo avatar Jul 01 '24 11:07 furilo

@andreslucena ready to review 😄

mllocs avatar Jul 03 '24 11:07 mllocs

@mllocs can you solve the git conflict 🙏🏽 ?

As I will not have the bandwidth to review PRs in the next few weeks, I'm assigning this review to @alecslupu.

andreslucena avatar Jul 17 '24 11:07 andreslucena