These changes make it so that downloading a VOID status CSV no longer causes an error. If status is void, we don't try to add the deleted comment info. Also, it updates to add the void status change to the report's history. This shows up in the report timeline on the single report page now.
These changes will impact:
[ ] commenters
[x] moderators
[x] admins
[ ] developers
What changes to the GraphQL/Database Schema does this PR introduce?
none
Does this PR introduce any new environment variables or feature flags?
no
If any indexes were added, were they added to INDEXES.md?
n/a
How do I test this PR?
You can create a User 1, have them comment. Then have another User 2 report their comment for illegal content. Then have the User 1 request their account be deleted. Go in and update their scheduled deletion date to be now. Run Coral with jobs so that they will be deleted (might want to update to make account deletion run more often in accountDeletion.ts so you don't have to wait). See that the user is successfully deleted with no errors. The illegal content report against their comment should now be set to status Void. Try to download the CSV for the report and see that there are no errors. Also see that the status update is shown in the report timeline (that the user deleted their account and the report is void).
Where any tests migrated to React Testing Library?
What does this PR do?
These changes make it so that downloading a
VOIDstatus CSV no longer causes an error. If status is void, we don't try to add the deleted comment info. Also, it updates to add the void status change to the report's history. This shows up in the report timeline on the single report page now.These changes will impact:
What changes to the GraphQL/Database Schema does this PR introduce?
none
Does this PR introduce any new environment variables or feature flags?
no
If any indexes were added, were they added to
INDEXES.md?n/a
How do I test this PR?
You can create a User 1, have them comment. Then have another User 2 report their comment for illegal content. Then have the User 1 request their account be deleted. Go in and update their scheduled deletion date to be now. Run Coral with jobs so that they will be deleted (might want to update to make account deletion run more often in accountDeletion.ts so you don't have to wait). See that the user is successfully deleted with no errors. The illegal content report against their comment should now be set to status Void. Try to download the CSV for the report and see that there are no errors. Also see that the status update is shown in the report timeline (that the user deleted their account and the report is void).
Where any tests migrated to React Testing Library?
How do we deploy this PR?