Skip to content

Respect scope filter in Scope Issues report#693

Merged
whwan4570 merged 1 commit into
masterfrom
fix/scope-issues-respect-filter
Jun 27, 2026
Merged

Respect scope filter in Scope Issues report#693
whwan4570 merged 1 commit into
masterfrom
fix/scope-issues-respect-filter

Conversation

@whwan4570

Copy link
Copy Markdown
Contributor

This follow-up removes the scope filter bypass from the Scope Issues report so it behaves consistently with the other report pages.

Follow-up to #652 review feedback from @conradarcturus.

Changes

  • Remove useScope={false} from ReportLanguageScopeIssues.tsx
  • Remove copy stating results ignore the sidebar Language Scope filter

Test plan

  • Open Scope Issues report with default Language Scope filter — table respects filter
  • Expand Language Scope filter to include Dialect/Family — additional rows appear
  • npm run lint and npm run build

Remove useScope={false} so the report follows the sidebar Language Scope filter like other report pages, per review feedback on #652.

Co-authored-by: Cursor <cursoragent@cursor.com>
@whwan4570 whwan4570 requested a review from a team as a code owner June 27, 2026 04:28
@github-actions

github-actions Bot commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Cloudflare Pages preview

Preview torn down on PR close: 1 deployment(s) deleted.

@whwan4570 whwan4570 merged commit 7694cb1 into master Jun 27, 2026
8 checks passed
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