Skip to content

Add PPI Validation to UI Workflow#4

Merged
Yuyuutsu merged 15 commits into
mainfrom
Add-PII-Validation-Display
Apr 29, 2026
Merged

Add PPI Validation to UI Workflow#4
Yuyuutsu merged 15 commits into
mainfrom
Add-PII-Validation-Display

Conversation

@Yuyuutsu
Copy link
Copy Markdown
Contributor

@Yuyuutsu Yuyuutsu commented Apr 22, 2026

Summary

This PR adds a PPI Validation functionality with a review step for flagged data.

Why

When the user uploads a CSV the survey-assist-pii processing must be triggered to check for possible sensitive data.

Changes

  • Added survey_assist_pii validation Package
  • Added /review_response route which displays flagged rows and requests user confirmation
  • Added /cancel when users abort the upload. This is point where the ppi json file report is built and saved
  • Added new tests and refactored old tests to account for PPI validation

How to Test

  1. Make sure Tests pass and new tests make sense
  2. Run the App Locally and compare it to the User Interface Workflow
  3. Check code changes make sense

@Yuyuutsu Yuyuutsu marked this pull request as ready for review April 28, 2026 10:21
@Yuyuutsu Yuyuutsu changed the title Add PPI Validation Functionality to UI Add PPI Validation to UI Workflow Apr 28, 2026
Comment thread cicd/cloudbuild_dev_and_sandbox.yaml
Copy link
Copy Markdown
Collaborator

@gibbardsteve gibbardsteve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gave it a quick scan, no real concerns. A minor comment to add something to the backlog to remind me to version the PII code and update the UI to use a tagged version.

Comment thread pyproject.toml
@Yuyuutsu Yuyuutsu merged commit 1187fca into main Apr 29, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants