Repository navigation
merge queue: checking #1919 on main (b69683d) - #1923
Closed
mergify[bot] wants to merge 7 commits into
Closed
mergify[bot] wants to merge 7 commits into
mergify[bot] wants to merge 7 commits into
Conversation
Cover a typed hidden path through the public App boundary: the URL is kept, the area title and German notice show, the settings link recovers the switch, and the flag persists across a remount. Unknown, visible and explicitly switched-off routes are controls. Plan-ID: GLASS-GI05-HIDE Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
A hidden area no longer redirects to the Leitstand. The URL stays, the shell shows the area title, a German notice and a link to Einstellungen, and ShellMount opens no fresh or legacy view underneath. The flag description now also names the gated Beweise and Core Steuerung. Test-First: src/shell/mount/scope-entry.mount.test.tsx::keeps a hidden deep link and shows its honest settings recovery Test-First: src/shell/mount/scope-entry.mount.test.tsx::leaves visible and unknown and explicitly switched-off routes as they are Test-First: src/shell/scope.test.tsx::with D1 on and the scope flag off the sidebar and palette omit non-shell areas and resolveRoute keeps the hidden route Test-First: src/shell/scope.test.tsx::with the scope flag off a hidden hash keeps its URL with a notice and settings link while /start stays Plan-ID: GLASS-GI05-HIDE Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
The legacy app synced its goal back into the hash when fed the default path for a hidden area. ShellMount now hands it the last visible route. Test-First: src/shell/mount/scope-entry.mount.test.tsx::keeps a hidden deep link after the legacy app has left another goal Plan-ID: GLASS-GI05-HIDE Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Plan-ID: GLASS-GI05-HIDE Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
The settings link in the hidden-area notice now carries the existing ghost button classes used by the version-updates screen. Href, text and routing are unchanged. No-Test: design class only, href and text unchanged, covered by existing src/shell tests Plan-ID: GLASS-GI05-HIDE Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Replace the stale redirect expectation with the approved hidden-area contract: retain the URL and show the honest note and settings link. Keep the accepted runtime and existing flag-enabled route walk unchanged. Plan-ID: GLASS-GI05-HIDE No-Test: test-only alignment to approved retained-URL behavior Co-authored-by: GPT-6.1 Sol <noreply@openai.com>
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
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.
Note
This is a merge queue draft opened by Mergify. Nobody needs to act on it.
Mergify opened it for batch diligent-painseigle-j6t067 of the
defaultqueue, to test #1919 before it merges intomain. Pushing to this branch or closing this pull request removes #1919 from the queue.What this draft tests
Mergify checks that #1919 still passes CI on top of
main. CI runs on this draft's code, built in this order:mainatb69683d🎉 Where it stands: checks passed
CI passed on this draft.
Follow the
defaultqueue in the dashboardWhat happens next
mainand closes this draft.Metadata for CI tooling, read by
mergify ciRequired conditions of queue rule
defaultfor mergecheck-success = gates (linux)check-success = gates (windows)check-success = red-firstgithub-review-approved[🛡 GitHub repository ruleset rulemain-pr-required-checks]label != do-not-mergemain-pr-required-checks]:check-success = @github-actions/gates (linux)check-neutral = @github-actions/gates (linux)check-skipped = @github-actions/gates (linux)main-pr-required-checks]:check-success = @github-actions/gates (windows)check-neutral = @github-actions/gates (windows)check-skipped = @github-actions/gates (windows)main-pr-required-checks]:check-success = @github-actions/red-firstcheck-neutral = @github-actions/red-firstcheck-skipped = @github-actions/red-firstmain-pr-required-checks]:check-neutral = @mergify/Mergify Merge Protectionscheck-skipped = @mergify/Mergify Merge Protectionscheck-success = @mergify/Mergify Merge ProtectionsRequired conditions to stay in the queue
-conflict-draftbase = maincheck-success = gates (linux)check-success = gates (windows)check-success = red-firstgithub-review-approved[🛡 GitHub repository ruleset rulemain-pr-required-checks]label != do-not-mergelabel = review-okmain-pr-required-checks]:check-success = @github-actions/gates (linux)check-neutral = @github-actions/gates (linux)check-skipped = @github-actions/gates (linux)main-pr-required-checks]:check-success = @github-actions/gates (windows)check-neutral = @github-actions/gates (windows)check-skipped = @github-actions/gates (windows)main-pr-required-checks]:check-success = @github-actions/red-firstcheck-neutral = @github-actions/red-firstcheck-skipped = @github-actions/red-firstmain-pr-required-checks]:check-neutral = @mergify/Mergify Merge Protectionscheck-skipped = @mergify/Mergify Merge Protectionscheck-success = @mergify/Mergify Merge Protections