Skip to content

fix(suite): Update category icons in guide#28236

Draft
jvaclavik wants to merge 1 commit into
developfrom
update-category-icons-in-guide
Draft

fix(suite): Update category icons in guide#28236
jvaclavik wants to merge 1 commit into
developfrom
update-category-icons-in-guide

Conversation

@jvaclavik
Copy link
Copy Markdown
Contributor

@jvaclavik jvaclavik commented Jun 1, 2026

Description

in category READMEs in trezor-suite-guide it matches:

<!-- icon: NAME -->

# example
<!-- icon: lifebuoy -->

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.ts
  • packages/suite/src/components/guide/GuideImage.tsx
  • packages/suite/src/components/guide/GuideRouter.tsx

Recommended 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.

⚠️ Changes with no test coverage (2)

  • packages/suite-data/src/guide/transformer.ts
  • packages/suite/src/components/guide/GuideImage.tsx

Updated: 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)
Test Type
Export transactions > Go to account and try to export all possible variants (pdf, csv, json) 🤖 auto
Account metadata > dropbox provider 🤖 auto
Passphrase with cardano > verify cardano address behind passphrase 🤖 auto
Cardano > Basic cardano walkthrough 🤖 auto
Public Keys > Check ada XPUB 🤖 auto
Account types suite > Add-account-types-non-BTC-coins 🤖 auto
Quarantine test: "Onboarding - create wallet,Success (basic)" 🙋 manual
Quarantine test: "Database migration,Db migration between: release/22.5/web => develop/web" 🙋 manual
Quarantine test: "Multiple sessions,Session overtaken by another" 🙋 manual
Quarantine test: "Recovery T2T1 - dry run,Recovery after partial recovery" 🙋 manual

Updated: 2026-06-01T13:10:38.416Z • 10 test(s) total

Trezor Suite (desktop) — 6 test(s)
Test Type
Passphrase with cardano > verify cardano address behind passphrase 🤖 auto
Cardano > Basic cardano walkthrough 🤖 auto
Public Keys > Check ada XPUB 🤖 auto
Account types suite > Add-account-types-non-BTC-coins 🤖 auto
Onboarding - create wallet > Success (basic) 🤖 auto
Quarantine test: "Multiple sessions,Session overtaken by another" 🙋 manual

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/

@jvaclavik jvaclavik changed the title fix(suite): Fix images on desktop build fix(suite): Update category icons in guide Jun 1, 2026
@jvaclavik jvaclavik force-pushed the update-category-icons-in-guide branch from ee27e0a to 08feb49 Compare June 1, 2026 15:54
@trezor-bot
Copy link
Copy Markdown
Contributor

trezor-bot Bot commented Jun 1, 2026

✅ Previously successful run of [Test] PR Suite Web e2e tests workflow has been found.
⏭️ Skipping tests for this run.
💡 If you are unsure about your latest changes, please rerun the workflow manually. (Use the Re-run all jobs option)

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.

1 participant