Skip to content

[MBL-19874][All] Privacy Consent dialog#3613

Open
hermannakos wants to merge 6 commits intomasterfrom
MBL-19874-privacy-consent-dialog
Open

[MBL-19874][All] Privacy Consent dialog#3613
hermannakos wants to merge 6 commits intomasterfrom
MBL-19874-privacy-consent-dialog

Conversation

@hermannakos
Copy link
Copy Markdown
Collaborator

refs: MBL-19874
affects: Student, Teacher, Parent
release note: Added consent screen for anonymous analytics.

test plan: Use beta. Check the ticket. Enable the cookie consent feature flag.

  1. Check if the consent screen appears on the first app open after the ff was enabled.
  2. Subsequent opens don't display the consent screen.
  3. The dialog is accessible via the Settings > Legal screen too.

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

test plan: You'll need local canvas to test as of now.
Go to each section that uses a webview, and navigate around, while checking the following:
There must be no console warning saying "OneTrust would initialize."
The window.JS_ENV object should contain MOBILE_COOKIE_CONSENT with a boolean true value.
Close the client.
Set the mobile_consent passed-on value to “false”.
Do the same as previously, MOBILE_COOKIE_CONSENT should show a boolean false value, and the console warning should not be visible.
refs: MBL-19874
affects: Student, Teacher, Parent
release note: Added consent screen for anonymous analytics.

test plan: Use beta. Check the ticket. Enable the cookie consent feature flag.
1. Check if the dialog appears on the first app open after the ff was enabled.
2. Subsequent opens don't display the dialog.
3. The dialog is accessible via the Settings > Legal screen too.
# Conflicts:
#	libs/pandautils/src/test/java/com/instructure/pandautils/domain/usecase/session/GetAuthenticatedSessionUseCaseTest.kt
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2026

🧪 Unit Test Results

✅ 📱 Parent App

  • Tests: 315 total, 0 failed, 0 skipped
  • Duration: 31.192s
  • Success Rate: 100%

✅ 📱 Student App

  • Tests: 1252 total, 0 failed, 0 skipped
  • Duration: 0.000s
  • Success Rate: 100%

✅ 📱 Teacher App

  • Tests: 373 total, 0 failed, 0 skipped
  • Duration: 30.476s
  • Success Rate: 100%

✅ 🌅 Horizon

  • Tests: 790 total, 0 failed, 0 skipped
  • Duration: 37.618s
  • Success Rate: 100%

✅ 📦 Submodules

  • Tests: 568 total, 0 failed, 0 skipped
  • Duration: 11.633s
  • Success Rate: 100%

📊 Summary

  • Total Tests: 3298
  • Failed: 0
  • Skipped: 0
  • Status: ✅ All tests passed!

Last updated: Tue, 07 Apr 2026 13:18:50 GMT

@github-actions
Copy link
Copy Markdown
Contributor

📊 Code Coverage Report

⚠️ Student

  • PR Coverage: 42.62%
  • Master Coverage: 42.65%
  • Delta: -0.03%

⚠️ Teacher

  • PR Coverage: 25.35%
  • Master Coverage: 25.37%
  • Delta: -0.02%

✅ Pandautils

  • PR Coverage: 23.74%
  • Master Coverage: 23.67%
  • Delta: +0.07%

📈 Overall Average

  • PR Coverage: 30.57%
  • Master Coverage: 30.57%
  • Delta: +0.01%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2026

Parent Install Page

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2026

Teacher Install Page

@hermannakos hermannakos requested a review from tamaskozmer April 7, 2026 13:10
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Student Install Page

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.

2 participants