This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Remove unnecessary use of jquery on the recaptcha page #11308
Labels
good first issue
Good for newcomers
S-Tolerable
Minor significance, cosmetic issues, low or no impact to users.
T-Task
Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Comments
squahtx
added
good first issue
Good for newcomers
S-Tolerable
Minor significance, cosmetic issues, low or no impact to users.
T-Task
Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
labels
Nov 11, 2021
Good day, How i submit my registration form?? |
I'll work on it! |
4 tasks
Hey, I would love to solve this issue. |
4 tasks
To do this:
|
Is this still needed? If so, can I work on it? |
Yes, feel free to work on it and open a PR. |
JeyRathnam
added a commit
to JeyRathnam/synapse
that referenced
this issue
Dec 13, 2022
JeyRathnam
added a commit
to JeyRathnam/synapse
that referenced
this issue
Dec 13, 2022
4 tasks
DMRobertson
pushed a commit
that referenced
this issue
Dec 14, 2022
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
good first issue
Good for newcomers
S-Tolerable
Minor significance, cosmetic issues, low or no impact to users.
T-Task
Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Currently we use jquery for some very rudimentary operations on the recaptcha page.
synapse/synapse/res/templates/recaptcha.html
Lines 8 to 13 in 73cbb28
This seems like overkill since they could easily be replaced by
document.getElementById
ordocument.querySelector
.The text was updated successfully, but these errors were encountered: