-
Notifications
You must be signed in to change notification settings - Fork 1k
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
fix: Allow all URL-safe characters in hidden field Id/ questionId #2250
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Ignored Deployments
|
Thank you for following the naming conventions for pull request titles! 🙏 |
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.
since this is a library file, please use lowercase validation.ts
for this file.
also this needs to be moved into the lib
folder instead of the components
folder :-)
@Dhruwang Looks good now 😊💪 Can you please solve the merge conflicts here? :-) |
…l-safe-characters-in-hidden-field
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.
@Dhruwang thanks for fixing this 😊 In the meantime there is also one more user request for this feature so I will merge it now to make everyone happy 😊🚀
What does this PR do?
Fixes 2010 #2332
Now underscore (_) and hyphen (-) is also allowed in hidden field Id / questionId
Also we were having separate logic for validating questionId and hidden field Id, added a common validation function for it
How should this be tested?
Use _ and - in hidden field Id and question Id
Checklist
Required
pnpm build
console.logs
git pull origin main
Appreciated