Skip to content
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

[#Issue 250] Delete PII for Users using TypeScript. #752

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

nickbennings
Copy link

Discord Username: @nickbennings

What type of PR is this? (select all that apply)

  • [βœ” ] πŸ• Feature
  • πŸ› Bug Fix
  • 🚧 Breaking Change
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ“ Documentation Update

**Description: **

The project contained personally identifiable information (PII) for users, which needed to be removed. While PII for users was successfully deleted, there was an exception for the GitHub token within the project on GitHub.

Related Tickets & Documents

QA Instructions, Screenshots, Recordings

This code was tested on a Linux laptop and cypress was run in the Firefox web browser.

UI accessibility concerns?

None

Added/updated tests?

  • πŸ‘ yes
  • [βœ” ] πŸ™… no, because they aren't needed
  • πŸ™‹ no, because I need help

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.

1 participant