Display only warnings of the currently selected scope by default - #6127
Draft
raphaelblum wants to merge 1 commit into
Draft
Display only warnings of the currently selected scope by default#6127raphaelblum wants to merge 1 commit into
raphaelblum wants to merge 1 commit into
Conversation
raphaelblum
force-pushed
the
warnings-current-scope-default
branch
from
August 6, 2026 07:37
15e2190 to
40720c0
Compare
raphaelblum
force-pushed
the
warnings-current-scope-default
branch
from
August 6, 2026 11:01
40720c0 to
c29156c
Compare
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.
Description
The warnings overview and the "Latest Warnings" dashboard widget query the warnings of every scope the user may access. For an admin that is every scope in the project, which makes the list hard to read — the ticket asks for the warnings of the current scope instead, without the user having to set a filter by hand.
WarningsGridandLatestWarningsDashboardWidgetnow pass the current content scope as thescopesquery variable instead of all allowed scopes. Switching the scope updates the list, because the query variable changes.Warnings that have no scope at all (DAM warnings, for instance) or only a part of a scope (only
domain) stay visible. The resolver already adds the partial scopes to the query and always matchesscope IS NULLnext to the requested scopes, so this needs no API change.WarningsPagealso passesshowAllScopesto itsContentScopeIndicator, so the page shows the current scope instead of "Global" unless all scopes are requested.Example
The new default needs no configuration. To keep the previous behavior, set the new
showAllScopesprop:Acceptance criteria
Screenshots/screencasts
Open TODOs/questions
showAllScopesrestores it in one line. If someone sees it differently, I can turn the prop around — keep all scopes as the default, let projects opt in to the current scope — and change the default in the next major instead.Further information
scopesquery variable in the admin rather than adding a second grid or a project-specific one.