Skip to content

Conversation

@imnasnainaec
Copy link
Collaborator

@imnasnainaec imnasnainaec commented Jan 20, 2026

Adds link to the Swagger UI in the AppBar, visible only when developer mode is enabled. Fits standard icon-only button patterns by using the IconButton component.

@imnasnainaec imnasnainaec added the 🟩Low Low-priority PR label Jan 20, 2026
@github-actions github-actions bot added the 💻 FW Lite issues related to the fw lite application, not miniLcm or crdt related label Jan 20, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

A new "Swagger" button is added to the developer actions section in the AppBar header, positioned alongside the existing Sandbox button. The Swagger button uses the i-mdi-api icon and opens in a new tab, with both buttons now including title attributes.

Changes

Cohort / File(s) Summary
Developer actions UI
frontend/viewer/src/home/HomeView.svelte
Added Swagger button next to Sandbox button in DevContent header using i-mdi-api icon (target="_blank"). Sandbox button now includes title attribute. Net change: +2/-1 lines.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

💻 FW Lite, 📦 Lexbox

Poem

🐰 A button so new, with swagger and grace,
To the toolbar it hops with API in place,
The Sandbox stands tall with a title so fine,
Two tools for the dev, in perfect align! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided by the author, making it impossible to evaluate whether it relates to the changeset. Add a pull request description explaining the purpose and context of adding the Swagger button for developers.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add Swagger button for devs' directly and clearly describes the main change: adding a new Swagger button to the developer actions area.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Jan 20, 2026

UI unit Tests

  1 files  ±0   47 suites  ±0   19s ⏱️ ±0s
123 tests ±0  123 ✅ ±0  0 💤 ±0  0 ❌ ±0 
184 runs  ±0  184 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 7888770. ± Comparison against base commit 0973431.

♻️ This comment has been updated with latest results.

@argos-ci
Copy link

argos-ci bot commented Jan 20, 2026

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ✅ No changes detected - Jan 20, 2026, 4:48 PM

@github-actions
Copy link

C# Unit Tests

146 tests  ±0   146 ✅ ±0   22s ⏱️ +4s
 22 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 7888770. ± Comparison against base commit 0973431.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@frontend/viewer/src/home/HomeView.svelte`:
- Around line 158-159: Update the two Button usages in HomeView.svelte so their
title attributes use the localization function ($t) instead of hardcoded strings
(e.g., replace title="Sandbox" and title="Swagger" with title={$t('...')} using
appropriate i18n keys), and add rel="noopener noreferrer" to the Button that
opens in a new tab (the Button with target="_blank" / icon="i-mdi-api") to
harden the external link.

@myieye myieye merged commit 601c05e into develop Jan 23, 2026
23 checks passed
@myieye myieye deleted the fwlite-add-swagger-link-for-devs branch January 23, 2026 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💻 FW Lite issues related to the fw lite application, not miniLcm or crdt related 🟩Low Low-priority PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants