Move the logic from @redux-devtools/app into @redux-devtools/app-core #1655
Conversation
🦋 Changeset detectedLatest commit: da2f235 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. |
|
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 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.
|
@markerikson I have rebased this now so it's mergable again. It would be good to get this reviewed and merged so I can switch my Expo devtools plugin away from using a fork of this package and use the official one. |
|
Thank you! Please let me know if there are any issues or anything you need to discuss. |
|
@matt-oakes The |
Themes.tsx could previously be imported at |
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.