Fix/2068#307
Open
conradmugabe wants to merge 4 commits into
Open
Conversation
Pulls in the enable_chat_history_export tenant setting (via @iblai/web-utils) that the chat-history export gate reads. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011tT29xrvXbGyeuokaWsPTg
Gate the chat-history export menu item on the new enable_chat_history_export tenant setting combined with the user's role: non-students always keep export; students lose it only when a tenant admin turns the setting off (default on). Reads the setting via useTenantMetadata and threads a canExport flag down to the row menu. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011tT29xrvXbGyeuokaWsPTg
Covers the tenant Advanced 'Chat History Export' switch (default on, persists) and the sidebar chat-row menu gating: student + setting off hides export, student + setting on shows it, non-student always sees it. Adds sidebar page-object helpers for the chat-row three-dot menu. Runs serial since all cases mutate the same tenant setting. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011tT29xrvXbGyeuokaWsPTg
Adds journey 56 (chat-history-export-toggle) with checkpoints chexp-01..05 to the e2e coverage manifest and doc. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011tT29xrvXbGyeuokaWsPTg
Member
|
Claude finished @conradmugabe's task —— View job E2E Coverage Review
Result: ✅ Approved — coverage files are correct and complete. Journey 56 (Chat History Export Toggle) adds 5 checkpoints, improving coverage from 499 → 504. All 25 app routes remain covered (100%). |
mamigot
approved these changes
Jul 4, 2026
mamigot
left a comment
mamigot
left a comment
Member
There was a problem hiding this comment.
✅ E2E Coverage — All Good
New e2e journey (56 — Chat History Export Toggle) is covered and coverage files are up to date.
Verified:
e2e/journeys/56-chat-history-export-toggle.spec.ts— 5 checkpoints addede2e/coverage.json— new journey entry with correctsourceFilesandcheckpointse2e/COVERAGE.md— matching[x]checkpoint lines added- Coverage improved: 499 → 504 checkpoints (+5), 100% coverage maintained
- Mechanical check: ✅ passed (exit code 0)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Checklist
Changes