Skip to content

web: Router prep for Command Palette#20723

Open
GirlBossRush wants to merge 4 commits intomainfrom
router-command-palette-prep
Open

web: Router prep for Command Palette#20723
GirlBossRush wants to merge 4 commits intomainfrom
router-command-palette-prep

Conversation

@GirlBossRush
Copy link
Contributor

Details

This PR prepares the router for use in the upcoming command palette (#20685).

The changes here are largely mechanical and align with the recent improvements to the flow executor.

Changes

  • A default export is added to every route handler
  • Lit's development warnings regarding @property setters defined without getters have been corrected.
  • Route definitions now have less repetition and use a nested structure to build out path patterns

@GirlBossRush GirlBossRush self-assigned this Mar 5, 2026
@GirlBossRush GirlBossRush requested a review from a team as a code owner March 5, 2026 01:06
@GirlBossRush GirlBossRush added the area:frontend Features or issues related to the browser, TypeScript, Node.js, etc label Mar 5, 2026
@netlify
Copy link

netlify bot commented Mar 5, 2026

Deploy Preview for authentik-storybook ready!

Name Link
🔨 Latest commit 1d84cc4
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/69a8d710ca88f00008065baf
😎 Deploy Preview https://deploy-preview-20723--authentik-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Mar 5, 2026

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 1d84cc4
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/69a8d7125cbfbe000907e61c
😎 Deploy Preview https://deploy-preview-20723--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.11%. Comparing base (ce1237e) to head (1d84cc4).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #20723      +/-   ##
==========================================
- Coverage   93.39%   93.11%   -0.28%     
==========================================
  Files         983      983              
  Lines       55477    55477              
==========================================
- Hits        51813    51658     -155     
- Misses       3664     3819     +155     
Flag Coverage Δ
conformance 37.38% <ø> (-0.31%) ⬇️
e2e 42.67% <ø> (-0.60%) ⬇️
integration 22.39% <ø> (-0.05%) ⬇️
unit 91.54% <ø> (+<0.01%) ⬆️
unit-migrate 91.63% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Features or issues related to the browser, TypeScript, Node.js, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant