-
Notifications
You must be signed in to change notification settings - Fork 74
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
Transparency Report email automation #1290
Transparency Report email automation #1290
Conversation
* The "Transparency Report Received" checkbox can only be checked or unchecked when the current user is admin or super admin. | ||
* See https://github.com/WordPress/wordcamp.org/issues/1280#issuecomment-2058571557. | ||
*/ | ||
if ( current_user_can( 'manage_options' ) ) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if you've tested this scenario but I've come across several times in the past where a user who isn't shown or displayed a field in this way can overwrite the field with a default value. Unsure if this is the case here, but wanted to mention it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for bringing that up! I hadn't thought of that case.
Just checked off the field and then updated the post using another user unable to see the checkbox, and it didn't overwrite.
Fixes #1280
This PR adds a checkbox for admins and super admins to check to confirm whether the transparency report has been received. And integrates with the organizer reminder plugin, allowing users to set how many days after a camp ends a reminder email should be sent if the report has not been received.
How to test the changes in this Pull Request:
organizing team (typically [email protected])
&The Budget Wrangler
.7
days &14 days
&21 days
.start date
,end date
, andthe Budget Wrangler's email address
; these can be entered arbitrarily. Also, fill in theURL
and check thecreate a new site
checkbox. Then Publish.Transparency Report Received
checkbox at the bottom of the WordCamp Information section.super admin
and anadmin+wordcamp
author can see the checkbox. (ie. make sure awordcamp author
can't see the checkbox)Need Vetting
toIn Pre-Planning
.End Date
is NOT set to 7 days ago. Then, runwp cron event run wcor_send_timed_emails --url=https://central.wordcamp.test/
in your docker container.End Date
of the second WordCamp for 7, 14, or 21 days; leave it for future testing.End Date
to 7 days ago and runwp cron event run wcor_send_timed_emails --url=https://central.wordcamp.test/
again.End Date
to 14 days and 21 days ago respectively and runwp cron event run wcor_send_timed_emails --url=https://central.wordcamp.test/
again.End Date
to 7 days ago. Then, runwp cron event run wcor_send_timed_emails --url=https://central.wordcamp.test/
in your docker container.Transparency Report Received
checkbox and set theEnd Date
to 14 days ago. Runwp cron event run wcor_send_timed_emails --url=https://central.wordcamp.test/
Screenshots
Organizer Reminder
Three emails received
Transparency Report Received Checkbox
Four emails received