Migrate anonymization script to react #10654
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
maxidragon
reviewed
Jan 19, 2025
...ponents/Tickets/TicketWorkbenches/AnonymizationTicketWorkbenchForWrt/PerformManualChecks.jsx
Outdated
Show resolved
Hide resolved
| remove the person's data. | ||
| </ListItem> | ||
| <ListItem> | ||
| For recently competed competitions (Past 1 month), after anonymizing the person's data, |
There was a problem hiding this comment.
Maybe it's worth mentioning that currently comps older than 3 months are removed automatically from Live?
There was a problem hiding this comment.
I'm not sure about that actually. Is that how it is happening? In that case, we need to do for past 3 months right?
There was a problem hiding this comment.
Yup, it was implemented recently.
There was a problem hiding this comment.
So that means "Past 1 month" needs to be replaced with "Past 3 months" and in brackets we can mention WCA Live will auto-clear after 3 months, am I right?
maxidragon
reviewed
Jan 19, 2025
kr-matthews
reviewed
Jan 21, 2025
app/webpacker/components/Panel/pages/AnonymizationScriptPage/index.jsx
Outdated
Show resolved
Hide resolved
app/webpacker/components/Tickets/TicketWorkbenches/AnonymizationTicketWorkbenchForWrt/index.jsx
Outdated
Show resolved
Hide resolved
danieljames-dj
force-pushed
the
anonym-script
branch
2 times, most recently
from
854423d to
5b3fe8c
Compare
danieljames-dj
force-pushed
the
anonym-script
branch
3 times, most recently
from
ffa78a5 to
db318a9
Compare
danieljames-dj
force-pushed
the
anonym-script
branch
from
db318a9 to
9170f04
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently there are two pages of anonymization - one for anonymizing profile (in rails) and another for anonymizing account (in react). This PR merges both to a single react script.