Skip to content

[MBL-19967][S/T/P] Privacy Consent - Logic update#4009

Merged
rh12 merged 3 commits into
masterfrom
feature/MBL-19967-Privacy-Consent-logic-update
Apr 27, 2026
Merged

[MBL-19967][S/T/P] Privacy Consent - Logic update#4009
rh12 merged 3 commits into
masterfrom
feature/MBL-19967-Privacy-Consent-logic-update

Conversation

@rh12
Copy link
Copy Markdown
Contributor

@rh12 rh12 commented Apr 24, 2026

PR Info

refs: MBL-19967
builds: Student, Teacher, Parent
affects: Student, Teacher, Parent
release note: none

What's new

See ticket for details.

Test Plan

The API is currently available only on beta env.

Testing on the PROD env, the tracking policy should fall back to the send_usage_metrics FF:

  • Verify when the FF is enabled tracking is enabled
  • Verify when the FF is disabled tracking is disabled
  • Verify Profile Menu > Settings shows no Privacy Settings in all 3 apps

Testing on the BETA env, the usage_metrics value can be governed by adjusting the FFs:

  1. Tracking always enabled
  • Set send_usage_metrics_after_consent FF to enabled
  • Set cookie_consent_necessary FF to disabled
  • (will result in: usage_metrics: "track_usage")
  • Verify no consent dialog is shown at login
  • Verify tracking is enabled (regardless of the above FF)
  • Verify Profile Menu > Settings shows no Privacy Settings row in all 3 apps
  1. Tracking always disabled
  • Set send_usage_metrics_after_consent FF to disabled
  • Set cookie_consent_necessary FF to disabled
  • (will result in: usage_metrics: "no_track_usage")
  • Verify no consent dialog is shown at login
  • Verify tracking is disabled (regardless of the above FF)
  • Verify Profile Menu > Settings shows no Privacy Settings row in all 3 apps
  1. Tracking requires consent
  • Set send_usage_metrics_after_consent FF to enabled
  • Set cookie_consent_necessary FF to enabled
  • (will result in: usage_metrics: "ask_for_consent")
  • Verify consent dialog is shown at login
  • Clear the user consent by logging out and in (or via the developer menu)
  • Verify tracking follows the choice (regardless of the above FF)
  • Verify Profile Menu > Settings shows Privacy Settings row in all 3 apps, and the value matches the choice
  • Verify Setting changes enabled/disable tracking as expected

Checklist

  • Follow-up e2e test ticket created
  • Tested on phone
  • Tested on tablet
  • Approve from product

@rh12 rh12 self-assigned this Apr 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

PR Checks

Commit: Allow internal tracking (ba31e32)
Built At: Apr 24 13:57 UTC (04/24 07:57 AM MDT)

✅ PR Description
✅ Copyright Headers
✅ SwiftLint
✅ Build CITests
✅ Unit Tests
✅ Code Coverage

@bitrise
Copy link
Copy Markdown
Contributor

bitrise Bot commented Apr 24, 2026

Builds

Commit: Allow internal tracking (ba31e32)
Build Number: 1892
Built At: Apr 24 16:05 CEST (04/24 08:05 AM MDT)

Student
Teacher
Parent

Copy link
Copy Markdown
Contributor

@petkybenedek petkybenedek left a comment

Choose a reason for hiding this comment

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

Code +1

Copy link
Copy Markdown
Collaborator

@vargaat vargaat left a comment

Choose a reason for hiding this comment

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

Code+1

Copy link
Copy Markdown
Contributor

@suhaibabsi-inst suhaibabsi-inst left a comment

Choose a reason for hiding this comment

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

Code +1

Copy link
Copy Markdown
Contributor

@petkybenedek petkybenedek left a comment

Choose a reason for hiding this comment

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

QA +1
All apps
macOS

Copy link
Copy Markdown
Collaborator

@vargaat vargaat left a comment

Choose a reason for hiding this comment

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

QA+1 for Student app

@rh12 rh12 merged commit f2ad0c7 into master Apr 27, 2026
9 checks passed
@rh12 rh12 deleted the feature/MBL-19967-Privacy-Consent-logic-update branch April 27, 2026 11:56
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