Move the logic from @redux-devtools/app into @redux-devtools/app-core #1655
Conversation
🦋 Changeset detectedLatest commit: 0a4089b The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
I have made the changes needed for the linting and tests to pass correctly. I've also added a changeset file which will set the new You should have push access, so feel free to make changes or comments as required. |
This change splits out the main logic from the Redux Devtools App into a new core package but keeps the socket connection management in @redux-devtools/app. The aim is to allow for easier reuse of the rest of the app in other envioronments with their own transport methods, such as React Native or Electron.
|
Whoops! I fixed the small formatting issue. Sorry, my editor had a different way to automatically format that and I forgot to run the check formatting locally. |
|
To test this out I have created an Expo Dev Plugin which makes use of this new It's working great and the amount of code needed to get the full devtools app UI with a custom connection middleware is now greatly reduced. |
This PR follows on from a previous discussion: #1653
This change splits out the main logic from the Redux Devtools App into a new core package but keeps the socket connection management in @redux-devtools/app. The aim is to allow for easier reuse of the rest of the app in other envioronments with their own transport methods, such as React Native or Electron.
The goal of this PR is that nothing will change for users of the
@redux-devtools/apppackage. It re-exports all of the exports from@redux-devtools/app-coreso everything that was previously available still is.The majority of the PR is just moving files and changes to imports. Some changes needed to be made to allow for some actions, reducers, and middleware to be kept in
@redux-devtools/apprather than moved to@redux-devtools/app-core. These are anything which relates to the management of the socket connection.There is also a new prop on the
Appcomponent which allows adding additional settings panels. This is used to add the "Connection" settings component, but not include it in the core package as it's specifically socket related. This poperty is optional to avoid any breaking changes.