-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Make spamaway ids unique #5465
Make spamaway ids unique #5465
Conversation
Generated by 🚫 Danger |
Hi! This looks great! Just because it's such a critical system, would you mind adding a screenshot of this working? Thank you! |
Hi @jywarren @gauravano I've updated the gif and information. |
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 @CleverFool77 for taking this issue! The change looks good, I just gave a few comments for style, a minor bug and an extra change that I think can be added.
Hi @alonpeer Thank you so much for your review. |
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.
Looks good! 💯
The failing test seems to be unrelated.
Hi @jywarren @gauravano |
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.
Great work!
Merged 🎉 💯 . Thanks @CleverFool77 @alonpeer! |
Fixes #4966 (<=== Add issue number here)
Description
Earlier , the callers for
create_form
template weremodal
and/signup
, the ids were only decided by the no ofspamaway
which was leading it to have non-unique ids.The approach to deal with it was -
I've passed local variables as the argument.
The argument tells us about the caller who is calling the
create_form
template which areloginmodal
form and another for/signup
form.The values passed from both of the callers is concatenated in spamaway ids. Thus making the ids unique as both the callers are passing different values.
The gif below shows that there is no more console error regarding spamaway ids.
Though I found there are some other errors for which I opened issue #5466
rake test
@publiclab/reviewers
for help, in a comment belowThanks!