-
Notifications
You must be signed in to change notification settings - Fork 383
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(react): native support react RSC #1914
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
size-limit report 📦
|
2493e08
to
7b416f4
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.
@thekip thank you!
@thekip let's ship it in v4? |
d9d94e2
to
315a5b2
Compare
Rebased and changed the base, this one #1915 is also important to be merged to make it work. |
@thekip could you please rebase the #1915 as well? There are some e2e tests failing btw |
Just found that splitting macro to two packages not in the main branch, so the fix #1915 is not needed right now. |
Also i have completely no idea how to test changes from this PR. Probably the only one way is full e2e test on nextjs application using cypress/playwright |
Probably this is the only way |
@thekip I'd like to make a new 4.9.0 release. Is there anything that is required for this release? |
@andrii-bodnar please, go ahead |
Description
Bring to the React package primitives for comfortable work with RSC
Related discussion: #1903 (comment)
Here is example using it: timofei-iatsenko/nextjs-lingui-rsc-poc#3
Types of changes
Fixes # (issue)
Checklist