feat(auth): show security warning dialog before account creation#700
Open
Vaishnavi10706 wants to merge 1 commit into
Open
feat(auth): show security warning dialog before account creation#700Vaishnavi10706 wants to merge 1 commit into
Vaishnavi10706 wants to merge 1 commit into
Conversation
- Add SecurityWarningModal component with title 'Security Warning' and message warning users not to reuse passwords from Google, Facebook, or other auth accounts - Intercept handleSubmit in SignUpScreenFirst and SignUpScreenSecond to show the warning before registration proceeds - User must tap 'Continue' to proceed or 'Cancel' to abort signup - Follows existing RN Modal pattern used by EmailVerifiedModal
Contributor
|
Thank you @, for creating the PR and contributing to our UltimateHealth project 💗. |
SB2318
requested changes
May 16, 2026
Owner
SB2318
left a comment
SB2318
left a comment
There was a problem hiding this comment.
Please attach one screenshot. Thanks for your contribution, Welcome to GSSOC'26
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds a security warning dialog that appears when a user attempts to create
an account. The user must acknowledge the warning before signup proceeds.
Fixes: #688
Changes Made
frontend/src/components/SecurityWarningModal.tsx(new file)A reusable modal component following the existing
EmailVerifiedModalpattern. Displays the security warning title, message, and two action
buttons Cancel and Continue.
frontend/src/screens/auth/SignUpScreenFirst.tsx(modified)Intercepts the submit action for both General User and Doctor flows.
Shows the security warning modal before any registration logic runs.
frontend/src/screens/auth/SignUpScreenSecond.tsx(modified)Intercepts the submit action for the Doctor registration flow.
Shows the security warning modal before calling the register API.
Behavior
What Was NOT Changed
Type of Change
Checklist