Skip to content

wrong vouching request email norification #376

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
Closed
hugosolar opened this issue Dec 12, 2019 · 1 comment · Fixed by #377
Closed

wrong vouching request email norification #376

hugosolar opened this issue Dec 12, 2019 · 1 comment · Fixed by #377
Assignees
@hugosolar
Milestone
High Priority

Comments

@hugosolar
Copy link
Contributor

There's seems to be a problem choosing members to be notified by email to review pending vouching request
@hugosolar hugosolar self-assigned this Dec 12, 2019
@hugosolar hugosolar added this to the High Priority milestone Dec 12, 2019
hugosolar added a commit that referenced this issue Dec 12, 2019
@hugosolar
fix #376 - Condition added to check if user is in vouching state
427c395
@hugosolar hugosolar mentioned this issue Dec 12, 2019
Merged
6 tasks
@hugosolar
Copy link
Contributor Author

There wasn't an error with the process, however, the amount of days of the pending vouching notifications seems to be a little bit aggressive because of that we changed these limit days to give more time to the member to respond to the vouching request.

Before

  • first reminder: 1st day
  • second reminder: 3rd day
  • third reminder: 5th day
  • Close request: 6th day

After

  • first reminder: 3rd day
  • second reminder: 6th day
  • third reminder: 8th day
  • Close request: 14th day

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hugosolar hugosolar

Labels
None yet
Projects
None yet
Milestone
High Priority
Development

Successfully merging a pull request may close this issue.

Less aggressive notification times creativecommons/commoners
1 participant
@hugosolar