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.

Sign up for GitHub

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

Jump to bottom

Improvements to validation notification #5944

Open
ramyaragupathy opened this issue Jun 26, 2023 · 1 comment · May be fixed by #5958
Open

Improvements to validation notification #5944

ramyaragupathy opened this issue Jun 26, 2023 · 1 comment · May be fixed by #5958
Assignees
@Aadesh-Baral
Labels
scope: backend

Comments

@ramyaragupathy
Copy link
Member

Is your feature request related to a problem? Please describe.
From a discussion on revamping notifications, - users see value in validation notification only if there is a comment/feedback added to the action. Otherwise, plain validated message is delivering no value

Describe the solution you'd like
Instead of sending a mail/notification on every validated task - send a mail/notification only if there is an additional comment added during the successful validation.

However, for invalidation of tasks, we should retain the current workflow as it ensures accountability and follow up from contributors
@ramyaragupathy
Copy link
Member Author

ramyaragupathy commented Jun 28, 2023 •
edited
Loading

Per discussion today:

  • Retain task validation/invalidation notifications within the TM
  • Modify the task validaton emails section:
    • instead of option for validation/invalidation emails, restrict the option for just invalidation emails
    • modify the subtext to Receive an email when a task you have contributed to requires more mapping.
image

@Aadesh-Baral Aadesh-Baral linked a pull request Jul 3, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Aadesh-Baral Aadesh-Baral

Labels
scope: backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Only send email alerts for task invalidation messages. hotosm/tasking-manager
2 participants
@ramyaragupathy @Aadesh-Baral