Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Flow main layout #2662

Merged
merged 12 commits into from
Aug 27, 2024
Merged

feat: Flow main layout #2662

merged 12 commits into from
Aug 27, 2024

Conversation

caalador
Copy link
Contributor

add withLayouts method to RouterConfigurationBuilder
for using Flow serverside view as main layout.
To enable call .withLayouts(Flow) in routes.tsx
and add "layout": true into the views ViewConfig.

Copy link

codecov bot commented Aug 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.28%. Comparing base (512117b) to head (614736b).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2662      +/-   ##
==========================================
+ Coverage   92.21%   92.28%   +0.07%     
==========================================
  Files          76       76              
  Lines        2466     2477      +11     
  Branches      639      643       +4     
==========================================
+ Hits         2274     2286      +12     
  Misses        144      144              
+ Partials       48       47       -1     
Flag Coverage Δ
unittests 92.28% <100.00%> (+0.07%) ⬆️

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.

Co-authored-by: Soroosh Taefi <[email protected]>
@caalador caalador force-pushed the feat/flow-main-layout branch 16 times, most recently from 5a562b5 to 87cdbab Compare August 22, 2024 06:26
@caalador caalador marked this pull request as ready for review August 26, 2024 04:50
Copy link
Contributor

@taefi taefi left a comment

Choose a reason for hiding this comment

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

Generally, looks fine, added a couple of nit picking comments ;)

packages/ts/file-router/src/types.d.ts Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Aug 27, 2024

Copy link
Contributor

@taefi taefi left a comment

Choose a reason for hiding this comment

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

LGTM!

@taefi taefi merged commit 1ffca32 into main Aug 27, 2024
15 checks passed
@taefi taefi deleted the feat/flow-main-layout branch August 27, 2024 06:40
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Hilla 24.5.0.alpha10 and is also targeting the upcoming stable 24.5.0 version.

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

Successfully merging this pull request may close these issues.

3 participants