Fixes search settings.spec.ts playwright failures - #28989
Conversation
❌ PR checklist incompleteThis PR cannot be merged until the following are addressed on its linked issue:
The fields live on the linked issue in the Shipping project (open the issue → right sidebar → Projects). After you set them, re-run this check (or push a commit) — issue/project changes do not re-trigger it automatically. Maintainers can bypass this check by adding the |
Code Review ✅ ApprovedResolves Playwright failures in searchSettings.spec.ts by removing obsolete column test cases and unused variables. No issues found. OptionsDisplay: compact → Showing less information. Comment with these commands to change:
Was this helpful? React with 👍 / 👎 | Gitar |
❌ UI Checkstyle Failed❌ Playwright — ESLint + Prettier + Organise ImportsOne or more Playwright test files have linting or formatting issues. ❌ Core Components — ESLint + PrettierOne or more core-component files have linting or formatting issues. Fix locally (fast — only checks files changed in this branch): make ui-checkstyle-changed |
|
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
|
🔴 Playwright Results — 1 failure(s), 22 flaky✅ 3473 passed · ❌ 1 failed · 🟡 22 flaky · ⏭️ 97 skipped
Genuine Failures (failed on all attempts)❌
|



Describe your changes:
Fixes #27946
Type of change:
High-level design:
N/A — small change.
Tests:
Use cases covered
Unit tests
Backend integration tests
Ingestion integration tests
Playwright (UI) tests
Manual testing performed
UI screen recording / screenshots:
Not applicable.
Checklist:
Fixes <issue-number>: <short explanation>Fixes #<issue-number>above.