-
Notifications
You must be signed in to change notification settings - Fork 56
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
feat: Flow main layout #2662
Conversation
rename layout String to flowLayout boolean
Codecov ReportAll modified and coverable lines are covered by tests ✅
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
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Co-authored-by: Soroosh Taefi <[email protected]>
5a562b5
to
87cdbab
Compare
87cdbab
to
ebac3be
Compare
There was a problem hiding this 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/test/runtime/RouterConfigurationBuilder.spec.tsx
Outdated
Show resolved
Hide resolved
packages/ts/file-router/src/runtime/RouterConfigurationBuilder.ts
Outdated
Show resolved
Hide resolved
Co-authored-by: Soroosh Taefi <[email protected]>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
This ticket/PR has been released with Hilla 24.5.0.alpha10 and is also targeting the upcoming stable 24.5.0 version. |
add
withLayouts
method to RouterConfigurationBuilderfor using Flow serverside view as main layout.
To enable call
.withLayouts(Flow)
inroutes.tsx
and add
"layout": true
into the viewsViewConfig
.