Account removal is broken #9820
Labels
Account deletion
Enabling users to delete account while preserving accountability for edits
🐛 bug
This is a bug, not a feature request.
JavaScript
🎯 P0
Comments
teolemon
added
Account deletion
|
I did a hotfix in prod with [% lang("delete_confirmation") | replace("'", "\'") %]. The complete fix is #9822 |
alexgarel
added a commit
that referenced
this issue
Mar 6, 2024
Translations coming from crowdin might contain single quotes or double quotes. So whenever we use lang in a json or javascript string we should escape quotes. I created edq and esq (Escape Single/Double Quotes) function for that (1st commit) and applied it in a lot of place (2nd commit). This should be the pattern to use from now on. fixes: - #9820
john-gom
pushed a commit
that referenced
this issue
May 24, 2024
Translations coming from crowdin might contain single quotes or double quotes. So whenever we use lang in a json or javascript string we should escape quotes. I created edq and esq (Escape Single/Double Quotes) function for that (1st commit) and applied it in a lot of place (2nd commit). This should be the pattern to use from now on. fixes: - #9820
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
One user told us he can't click on the remove my account button.
The user.pl javascript seems broken.
If the console:
Uncaught SyntaxError: missing ) after argument listThe text was updated successfully, but these errors were encountered: