Skip to content

Purge user PII data#40683

Merged
hacodeorg merged 5 commits into
stagingfrom
ha/purge-user-data
May 25, 2021
Merged

Purge user PII data#40683
hacodeorg merged 5 commits into
stagingfrom
ha/purge-user-data

Conversation

@hacodeorg

@hacodeorg hacodeorg commented May 20, 2021

Copy link
Copy Markdown
Contributor

FND-1417

Data to purge

  • Contact rollups data in ContactRollupsFinal.
  • Survey data in Pd::MiscSurvey, Foorm::Submission, and Foorm::SimpleSurveySubmission.
  • Professional development email history in Pd::Application::Email.

No action needed

  • The current process already purges TeacherFeedback and ContactRollupsPardotMemory.
  • ContactRollupsRaw and ContactRollupsProcessed are just intermediate tables. They are dropped and rebuilt everyday by the contact rollups process..
  • contact_rollups_comparison table was dropped long ago.
  • Pd::Application::Tag model doesn't contain PII data. The name field is just tag name. In addition, the pd_application_tags table is currently empty.
  • Donor model doesn't contain PII data. It has organization/donor names, URLs and Twitter links; all can be considered public information. Moreover, the donors table has only 166 records. They can be manually deleted upon request.

Links

Testing story

  • Run unit tests bundle exec rails test test/helpers/delete_accounts_helper_test.rb

Follow-up work

  • Drop survey_answers table. Last created row was from 2019-10-22. We have migrated previous survey data to the new foorm system.
  • Back-purging data in ContactRollupsFinal, ContactRollupsPardotMemory, Pd::MiscSurvey, Foorm::Submission, Foorm::SimpleSurveySubmission, and Pd::Application::Email.

PR Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@hacodeorg hacodeorg force-pushed the ha/purge-user-data branch from f204e50 to 1bb5417 Compare May 20, 2021 09:16
@hacodeorg hacodeorg changed the title Purge user data Purge user PII data May 20, 2021
@hacodeorg hacodeorg marked this pull request as ready for review May 20, 2021 18:20
@hacodeorg hacodeorg requested review from a team, bencodeorg and sureshc May 20, 2021 18:20
Comment thread lib/cdo/delete_accounts_helper.rb
Comment thread dashboard/test/helpers/delete_accounts_helper_test.rb
Comment thread dashboard/test/helpers/delete_accounts_helper_test.rb
@hacodeorg hacodeorg merged commit 9c7b857 into staging May 25, 2021
@hacodeorg hacodeorg deleted the ha/purge-user-data branch May 25, 2021 18:35
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.

4 participants