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
feat: Extend Prefilling with an option to auto-skip prefilled values #2598
feat: Extend Prefilling with an option to auto-skip prefilled values #2598
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! 🙏 |
…145-extend-prefilling-with-an-option-to-auto-skip-prefilled-values
…145-extend-prefilling-with-an-option-to-auto-skip-prefilled-values
…lled-values' of https://github.com/formbricks/formbricks into 145-extend-prefilling-with-an-option-to-auto-skip-prefilled-values
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 a lot for the PR! @gupta-piyush19 😊
Looks good, just left some comments for minor refactor, pls take a look 🙏
…145-extend-prefilling-with-an-option-to-auto-skip-prefilled-values
…lled-values' of https://github.com/formbricks/formbricks into 145-extend-prefilling-with-an-option-to-auto-skip-prefilled-values
…-skip-prefilled-values
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 a lot for the changes!
LGTM! 🚀
…145-extend-prefilling-with-an-option-to-auto-skip-prefilled-values
…lled-values' of https://github.com/formbricks/formbricks into 145-extend-prefilling-with-an-option-to-auto-skip-prefilled-values
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.
@gupta-piyush19 Nice, the feature works great!! :-) 💪
I only changed the value of skipPrefilling
from 1
to true
since I noticed we now also use true
in all other functionality and this felt more consistent with the other features.
Great job; let's merge this :-)
What does this PR do?
Extend Prefilling with an option to auto-skip prefilled values.
Fixes
https://github.com/formbricks/internal/issues/145
How should this be tested?
skipPrefilled=1
.Checklist
Required
pnpm build
console.logs
git pull origin main
Appreciated