fix(suite): Update category icons in guide#28236
Draft
jvaclavik wants to merge 1 commit into
Draft
Conversation
ee27e0a to
08feb49
Compare
Contributor
|
✅ Previously successful run of [Test] PR Suite Web e2e tests workflow has been found. |
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
in category READMEs in
trezor-suite-guideit matches:Notes for QA
Related Issue
Resolve
Screenshots:
🤖 LLM Test Recommendations
Summary: The changes are concentrated in the Suite Guide feature: GuideRouter.tsx (guide panel navigation/routing), GuideImage.tsx (image rendering in guide articles), and transformer.ts (guide content transformation/processing). The highest risk is breaking guide panel navigation, content display, and search functionality. GuideRouter.tsx maps to 121 tests via static analysis, but this is because it's likely bundled as part of the main app layout — only the guide-specific tests are meaningfully affected. The recommended strategy is to run the two guide-specific tests at high priority, the bug report form test at medium (since it navigates through the guide panel), and optionally the link checker at low priority.
Changed files (3)
packages/suite-data/src/guide/transformer.tspackages/suite/src/components/guide/GuideImage.tsxpackages/suite/src/components/guide/GuideRouter.tsxRecommended tests (4)
🔴 High priority (2)
suite/e2e/tests/general/suite-guide.test.ts— This test directly exercises the Suite Guide panel, including opening it, navigating articles, and searching for content. The changed files GuideRouter.tsx (routing within the guide panel) and GuideImage.tsx (image rendering in guide content) are core components of this feature. The transformer.ts change affects how guide content is processed for display.suite/e2e/tests/suite/guide.test.ts— This test comprehensively covers guide panel behaviors: opening/closing, navigating content nodes, viewing article details, searching for topics, and submitting feedback. GuideRouter.tsx directly controls guide navigation and content rendering, and GuideImage.tsx affects how guide images display within articles.🟡 Medium priority (1)
suite/e2e/tests/suite/bug-report-form.test.ts— This test opens the Guide panel and navigates to the Support and Feedback section to submit a bug report. GuideRouter.tsx controls the routing within the guide panel, so changes there could affect navigation to the feedback section.🟢 Low priority (1)
suite/e2e/tests/general/check-links.test.ts— This test crawls all application routes and checks links for validity. If the guide transformer changes how links are generated or rendered in guide content, broken links could be introduced. This is a plausible but indirect impact.packages/suite-data/src/guide/transformer.tspackages/suite/src/components/guide/GuideImage.tsxUpdated: 2026-06-01T13:03:21.972Z
🔍 Currents Test Results
🔍 Suite desktop test results: View in Currents
🔍 Suite web test results: View in Currents
🔍 Suite native android test results: View in Currents
🔒 Quarantined E2E Tests
Trezor Suite (web) — 10 test(s)
Updated: 2026-06-01T13:10:38.416Z • 10 test(s) total
Trezor Suite (desktop) — 6 test(s)
Updated: 2026-06-01T15:59:36.356Z • 6 test(s) total
🌐 Preview deployments
🌐 Suite Web preview: https://dev.suite.sldev.cz/suite-web/update-category-icons-in-guide/web/