Skip to content

Fix/assessment mode fullscreen mode informative popover#166

Merged
sonegillis1 merged 3 commits into
mainfrom
fix/assessment-mode-fullscreen-mode-informative-popover
Jul 17, 2026
Merged

Fix/assessment mode fullscreen mode informative popover#166
sonegillis1 merged 3 commits into
mainfrom
fix/assessment-mode-fullscreen-mode-informative-popover

Conversation

@michael-on-code

Copy link
Copy Markdown
Contributor

Checklist

  • Tests were added/updated according to the feature/bugfix/change made
  • Version was rolled according to semver requirements
  • API endpoints openapi schema was updated if applicable

Changes

  • Fix/assessment mode fullscreen mode informative popover

@mamigot

mamigot commented Jul 17, 2026

Copy link
Copy Markdown
Member

Claude finished @michael-on-code's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews
  • Identify changed files
  • Run mechanical coverage check
  • Read current coverage state
  • Apply decision logic and post review

✅ E2E Coverage — No Issues

Changed files:

  • app/platform/[tenant]/course-content/[course_id]/__tests__/layout.test.tsx
  • app/platform/[tenant]/course-content/[course_id]/layout.tsx
  • components/edx-iframe/__tests__/edx-iframe.test.tsx
  • components/edx-iframe/edx-iframe.tsx
  • components/ui/__tests__/popover.test.tsx
  • components/ui/popover.tsx

No new page.tsx routes were added. Coverage files (e2e/coverage.json, e2e/COVERAGE.md) are unchanged — no regression detected.

Decision: CASE E — No new app routes were introduced. All existing coverage is intact.

Note: The gh pr review --approve command requires explicit tool approval in this environment. A formal GitHub review approval could not be posted, but coverage analysis confirms no issues with this PR.


@sonegillis1
sonegillis1 merged commit 4e6789e into main Jul 17, 2026
14 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants