Skip to content
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

Update to redux devtools cli #30

Closed
wants to merge 4 commits into from
Closed

Update to redux devtools cli #30

wants to merge 4 commits into from

Conversation

nickmeinhold
Copy link

@nickmeinhold nickmeinhold commented Aug 5, 2020

Just the documentation changes for #29

  • app is currently not connecting to the server
  • created a draft PR and will look into why an app won't connect to the server if time permits

EDIT:

  • added notes to README on changing the settings in the Redux DevTools window so an app will connect
  • changed the PR from draft to "ready for review" as I won't keep looking into it for now because the redux devtools cli is working for me (although changing the settings isn't ideal)

@coveralls
Copy link

coveralls commented Aug 5, 2020

Pull Request Test Coverage Report for Build 135

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 92.308%

Totals Coverage Status
Change from base Build 132: 0.0%
Covered Lines: 84
Relevant Lines: 91

💛 - Coveralls

@MichaelMarner
Copy link
Owner

Hey just an update on this - it looks as though it is currently not possible to specify the host/port when starting devtools-cli - meaning you need to manually edit the connection settings after it is running. I think until that issue is resolved we'll hold off on merging this PR.

Not being able to specify the host/port from the command line leads to a poorer developer experience, IMO.

@nickmeinhold
Copy link
Author

Thanks for the update @MichaelMarner. I think there are currently also some other problems with the redux-devtools-cli eg. I've noticed switching between Action and State while the app is running doesn't always update the Inspector's view. I will make an issue for that at some stage and report back. Thanks!

@Methuselah96
Copy link

Methuselah96 commented Aug 20, 2020

@MichaelMarner To be clear, the server is correctly started at the correct hostname and port, it's just that the Electron app that is opened doesn't automatically connect to that hostname and port.

You can also still go to http://localhost:8000 in your web browser (with or without the --open flag) if you prefer to do it that way, so I'm not sure that it's a poorer developer experience since you can still do it the old way without problems. (Or you can start it --open=browser and it will open the browser at the correct URL as well.) That being said, redux-devtools-cli is definitely still pre-release so no worries on holding off switching the documentation.

I'll look into automatically configuring the Electron app with the correct hostname and port when I get a chance.

@nickmeinhold Definitely would appreciate bug reports. Thanks for using it!

@nickmeinhold nickmeinhold closed this by deleting the head repository Sep 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants