Skip to content

feat: Added Pin Protection to forms #1142

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.

Sign up for GitHub

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

Jump to bottom
Merged
mattinannt merged 39 commits into from
Oct 16, 2023
Merged

feat: Added Pin Protection to forms #1142

mattinannt merged 39 commits into from
Oct 16, 2023

Conversation

@piyushgarg-dev
Copy link
Contributor

@piyushgarg-dev piyushgarg-dev commented Oct 14, 2023 •
edited
Loading

What does this PR do?

This PR adds a feature to protect the surveys using a 4-digit PIN and viewers need to enter a PIN to record a response.

Fixes #1089

Forbricks_PIN.mp4

Type of change

  • New feature (non-breaking change which adds functionality)

How should this be tested?

  • Create a new Form / Survey
  • Go to Settings and expand the Response Options
  • Toggle the PIN option
  • Add PIN to Survey

Checklist

Required

  • Filled out the "How to test" section in this PR
  • Read How we Code at Formbricks
  • Self-reviewed my own code
  • Commented on my code in hard-to-understand bits
  • Ran pnpm build
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR
  • Updated the Formbricks Docs if changes were necessary

@vercel
Copy link

vercel bot commented Oct 14, 2023

@piyushgarg-dev is attempting to deploy a commit to the formbricks Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added ❗️ migrations enhancement New feature or request formtribe-2023 hacktoberfest complete these issues to gather points for Hacktoberfest labels Oct 14, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 14, 2023 •
edited
Loading

Thank you for following the naming conventions for pull request titles! 🙏

@jobenjada jobenjada requested a review from mattinannt October 14, 2023 14:58
@jobenjada
Copy link
Member

jobenjada commented Oct 14, 2023 •
edited
Loading

Works really well, thanks a lot! :)

Matti will merge it as soon as our merge pipeline is back up and running 🚀

@mattinannt
Copy link
Member

mattinannt commented Oct 15, 2023

@piyushgarg-dev thanks a lot for this feature and the documentation of it on youtube :-)
You used the npm package react-otp-input for this. We already have built an OTP input for our 2 factor authentication feature (packages/ui/OTPInput/index.tsx). Is it possible that you use this for this feature as well?
If so, please change the code base to use this to reduce frontend packages and avoid an increase in bundle size.

impolska742 and others added 7 commits October 16, 2023 09:47
@impolska742 @jobenjada @piyushgarg-dev
fix: added scroll to top button (#1153)
b9b2ce0 
Co-authored-by: Johannes <johannes@formbricks.com>
@DonKoko @jobenjada @piyushgarg-dev
chore: Add shelf.nu to oss fields api endpoint (#1110)
b554e2d 
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
@itzabhinavarya @jobenjada @piyushgarg-dev
fix: twitter-icon-in-docs-footer-section (#1134)
0090af4 
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
@Sha1kh4 @jobenjada @piyushgarg-dev
fix: Added DISCORD to the Footer.tsx (#1154)
fd0eada 
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
@asharan2511 @jobenjada @piyushgarg-dev
fix/Typo-main-page (#1146)
79df5f5 
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
@mattinannt @piyushgarg-dev
chore: improve logging in survey service (#1156)
d9d2ae8
@aryabyte21 @jobenjada @piyushgarg-dev
fix: Automate copyright year (#1136)
1e36be9 
Signed-off-by: aryabyte21 <arya2001bhosale@gmail.com>
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
mattinannt and others added 16 commits October 16, 2023 09:48
@mattinannt @piyushgarg-dev
fix: build issues in environment service on some systems (#1160)
3faa54a
@ratishjain12 @piyushgarg-dev
fix: Concierge btn text opacity fixed and made it look proper (#1162)
8ea47d5
@sundaram2021 @piyushgarg-dev
fix: Pricing Link in mobile menu on LP , Update Header.tsx (#1163)
343e206
@Sha1kh4 @piyushgarg-dev
style: Fixed a minor grammatical mistake. (#1164)
d315306
@aryabyte21 @jobenjada @piyushgarg-dev
fix: minor ui fixes (#1138)
3ea0fd2 
Signed-off-by: aryabyte21 <arya2001bhosale@gmail.com>
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
@sohamtembhurne @jobenjada @piyushgarg-dev
fix: Link for "Pricing" not working fixed (#1165)
220e5a0 
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
@3xp10it3r @jobenjada @piyushgarg-dev
fix: Fixed grammatical error in feedback box -> why it is useful sect…
33609c0 
…ion (#1166)

Co-authored-by: Digvijay Gupta <digvijaygupta@Digvijays-MacBook-Pro.local>
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
@rtpa25 @jobenjada
@mattinannt @piyushgarg-dev
feat: adds hidden field functionality to surveys (#1144)
bf32cef 
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
Co-authored-by: Matthias Nannt <mail@matthiasnannt.com>
@bilalmirza74 @jobenjada @piyushgarg-dev
fix: Overflow of body across leaderboard on mobile screen #1120 & Doc…
d50b9c1 
…'s navbar was not responsive #1129 (#1123)

Co-authored-by: Johannes <johannes@formbricks.com>
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
@piyushgarg-dev
refactor: re-use OTPInput and removed react-otp-input
fadde9b
@mattinannt @piyushgarg-dev
fix: build issues in environment service on some systems (#1160)
4c05937
@sohamtembhurne @jobenjada @piyushgarg-dev
fix: Link for "Pricing" not working fixed (#1165)
df94088 
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
@3xp10it3r @jobenjada @piyushgarg-dev
fix: Fixed grammatical error in feedback box -> why it is useful sect…
788d04f 
…ion (#1166)

Co-authored-by: Digvijay Gupta <digvijaygupta@Digvijays-MacBook-Pro.local>
Co-authored-by: Johannes <72809645+jobenjada@users.noreply.github.com>
@piyushgarg-dev
Merge remote-tracking branch 'upstream/main' into feat/pin-protect-su…
4d000fc 
…rveys
@piyushgarg-dev
rever: pnpm-lock.yaml
9732ee8
@piyushgarg-dev
revert: pnpm-lock.yaml
678e2c1
@piyushgarg-dev
Copy link
Contributor Author

piyushgarg-dev commented Oct 16, 2023

Hi @mattinannt 👋🏻
Thanks for the feedback. I have re-used the packages/ui/OTPInput/index.tsx component for the PIN input. Also, I have added one extra optional prop for disabled.

Thanks 🙏🏻

@mattinannt mattinannt enabled auto-merge October 16, 2023 11:07
@mattinannt
Copy link
Member

mattinannt commented Oct 16, 2023

@piyushgarg-dev thanks a lot for making the changes :-) 💪🎉
I only update a few smaller things to follow current code conventions better. I will merge the feature into main now :-)

@mattinannt mattinannt added this pull request to the merge queue Oct 16, 2023
Merged via the queue into formbricks:main with commit 5e5723d Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mattinannt mattinannt mattinannt approved these changes

Assignees

@mattinannt mattinannt

Labels

enhancement New feature or request hacktoberfest complete these issues to gather points for Hacktoberfest ❗️ migrations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FEATURE][FormTribe 🔥][500 Points] PIN protected surveys