Skip to content

Commit

Permalink
Move fields to the hidden pii config
Browse files Browse the repository at this point in the history
Data and Insights have informed us that BigQuery is now allowed to
handle storing PII. This commit moves the requested fields from the
block list to the hidden_pii analytics config to allow them to be synced
over.
  • Loading branch information
rjlynch committed Jan 22, 2025
1 parent 3adf228 commit 7dc44e8
Show file tree
Hide file tree
Showing 4 changed files with 64 additions and 37 deletions.
28 changes: 28 additions & 0 deletions config/analytics.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,7 @@ shared:
- logged_in_with_onelogin
- onelogin_auth_at
- onelogin_idv_at
- onelogin_uid
- started_at
- verified_at
:decisions:
Expand Down Expand Up @@ -160,6 +161,7 @@ shared:
- had_leadership_position
- mostly_performed_leadership_duties
- claim_school_somewhere_else
- teacher_reference_number
:early_career_payments_eligibilities:
- id
- nqt_in_academic_year_after_itt
Expand All @@ -178,6 +180,7 @@ shared:
- award_amount
- induction_completed
- school_somewhere_else
- teacher_reference_number
:levelling_up_premium_payments_eligibilities:
- id
- created_at
Expand All @@ -197,6 +200,7 @@ shared:
- eligible_degree_subject
- induction_completed
- school_somewhere_else
- teacher_reference_number
:international_relocation_payments_eligibilities:
- id
- created_at
Expand Down Expand Up @@ -306,6 +310,7 @@ shared:
- flagged_as_duplicate
- provider_verification_email_last_sent_at
- provider_verification_chase_email_last_sent_at
- teacher_reference_number
:eligible_fe_providers:
- id
- ukprn
Expand All @@ -327,3 +332,26 @@ shared:
- updated_at
- provider_claim_submitted_at
- practitioner_claim_started_at
:school_workforce_censuses:
- id
- teacher_reference_number
- school_urn
- totfte
- contract_agreement_type
- subject_description_sfr
- general_subject_code
- hours_taught
- created_at
- updated_at
:teachers_pensions_service:
- id
- teacher_reference_number
- start_date
- end_date
- la_urn
- school_urn
- employer_id
- nino
- created_at
- updated_at
- gender_digit
35 changes: 0 additions & 35 deletions config/analytics_blocklist.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,15 +31,10 @@
- paye_reference
- practitioner_email_address
- provider_contact_name
- onelogin_uid
- onelogin_idv_first_name
- onelogin_idv_last_name
- onelogin_idv_full_name
- onelogin_idv_date_of_birth
:claim_decisions:
- trn
- claimant_age
- claimant_gender
:delayed_jobs:
- id
- priority
Expand Down Expand Up @@ -95,40 +90,10 @@
- amount
- created_at
- updated_at
:school_workforce_censuses:
- id
- teacher_reference_number
- school_urn
- totfte
- contract_agreement_type
- subject_description_sfr
- general_subject_code
- hours_taught
- created_at
- updated_at
:teachers_pensions_service:
- id
- teacher_reference_number
- start_date
- end_date
- la_urn
- school_urn
- employer_id
- nino
- created_at
- updated_at
- gender_digit
:international_relocation_payments_eligibilities:
- passport_number
- school_headteacher_name
:levelling_up_premium_payments_eligibilities:
- teacher_reference_number
:early_career_payments_eligibilities:
- teacher_reference_number
:student_loans_eligibilities:
- teacher_reference_number
:further_education_payments_eligibilities:
- teacher_reference_number
- verification
:journeys_sessions:
- answers
Expand Down
35 changes: 34 additions & 1 deletion config/analytics_hidden_pii.yml
Original file line number Diff line number Diff line change
@@ -1,2 +1,35 @@
---
shared: {}
:shared:
:claims:
- onelogin_uid
:levelling_up_premium_payments_eligibilities:
- teacher_reference_number
:early_career_payments_eligibilities:
- teacher_reference_number
:student_loans_eligibilities:
- teacher_reference_number
:further_education_payments_eligibilities:
- teacher_reference_number
:school_workforce_censuses:
- id
- teacher_reference_number
- school_urn
- totfte
- contract_agreement_type
- subject_description_sfr
- general_subject_code
- hours_taught
- created_at
- updated_at
:teachers_pensions_service:
- id
- teacher_reference_number
- start_date
- end_date
- la_urn
- school_urn
- employer_id
- nino
- created_at
- updated_at
- gender_digit
3 changes: 2 additions & 1 deletion spec/lib/analytics_importer_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@
"event_type" => "import_entity",
"entity_table_name" => "claims",
"event_tags" => ["20240101000000"],
"data" => a_hash_including({"key" => "id", "value" => [claim.id]})
"data" => a_hash_including({"key" => "id", "value" => [claim.id]}),
"hidden_data" => [{"key" => "onelogin_uid", "value" => []}]
}
],
ignore_unknown: true
Expand Down

0 comments on commit 7dc44e8

Please sign in to comment.