-
Notifications
You must be signed in to change notification settings - Fork 14
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: implement experimental support for server side rendering (SSR) #2466
Conversation
|
fd8e111
to
0eb7315
Compare
c9d5dc5
to
361848c
Compare
…aries BREAKING CHANGE: The `SbbDatepicker` property `selectedDate` has been renamed to `selected`. This also applies to the attribute `selected-date`, which has been renamed to `selected`. Additionally the `DateAdapter` (and `NativeDateAdapter`) have been superficially refactored. An important change is that the month is now `1`-based, instead of `0`-based.
361848c
to
2991f01
Compare
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #2466 +/- ##
=======================================
Coverage ? 93.64%
=======================================
Files ? 293
Lines ? 24174
Branches ? 1989
=======================================
Hits ? 22638
Misses ? 1507
Partials ? 29 ☔ View full report in Codecov by Sentry. |
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 PR adds the experimental possibility to use server side rendering with
@sbb-esta/lyne-components
. Most components have been verified to work with SSR, but our testing setup is not yet complete and will be finished in a future PR.