Bug/10354 - In-App Feedback: Phone number, SSN, Email are not properly recognized if they end in punctuation #10395
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 of Change
Ticket
The in-app feedback screen doesn't allow for personal information such as phone numbers, SSN and emails, if they are detected then we must show the user a warning.
As described in the ticket, if someone input an email but ended it with punctuation then we wouldn't show the user the warning.
This PR fixes that issue, we can now detect sensitive information even if it has punctuation.
Screenshots/Video
Testing
Navigate to the developer screen to find the in-app feedback screen
Reviewer Validations
Verify that sensitive information like emails, phone numbers and SSNs that end in punctuation trigger a warning alert.
EX:
PR Checklist
Reviewer: Confirm the items below as you review
For QA
Run a build for this branch