-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
v2.0 Integration branch #3089
v2.0 Integration branch #3089
Conversation
Open in CodeSandbox Web Editor | VS Code | VS Code Insiders |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 0fa46d4:
|
✅ Deploy Preview for redux-starter-kit-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Hi Mark, I've seen that you're migrating Toolkit to type: module. It seems well done but I have a question: do you need to publish one by one do you just need to publish the top level package (Toolkit)? |
@FaberVitale : there's only one actual package - Also, I had a couple meetings last week with Andarist and with a TS team member, who gave me advice on what we should do for proper ESM support. Notably, one suggestion is to drop |
1348302
to
157536c
Compare
Begin updating docs for 2.0 changes
- Updated title - Changed "NextJS" to "Next.js" to match their naming - Added recommendation on folder structure - Updated examples to match that folder structure - Added typed hooks setup and usage - Renamed `StoreType` to `AppStore` - Updated store init to always use `if (!storeRef.current)` - Updated section headers - Added "RTKQ doesn't do SSR yet" callout
click "LGTM" 🚢 🚢 |
This is an ongoing integration branch for all of the RTK 2.0 development work.
(second attempt, because Github accidentally closed the other one.)