Skip to content

Conversation

@smacpherson64
Copy link
Contributor

With the redux-devtools-cli, currently when using the --open flag the host and protocol options are ignored and opens http://localhost (causing a blank screen).

This PR adjusts --open to use the provided protocol and host options when possible.

Currently when using open, the cli will ignore the host and protocol and always open `http://localhost`.

This pr adjusts the open script to use the options protocol and host and default back to localhost if not provided.
@changeset-bot
Copy link

changeset-bot bot commented Aug 24, 2022

🦋 Changeset detected

Latest commit: 2cd9cbb

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@redux-devtools/cli Patch

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

@smacpherson64 smacpherson64 changed the title Adjusts the location open uses Adjusts @redux-devtools/cli --open flag to respect protocol and host Aug 24, 2022
@smacpherson64
Copy link
Contributor Author

Hi @Methuselah96, sorry for pinging! I was wondering what the process is about pull requests for @redux-devtools/cli. Is there anything that needs to be changed for this specific PR to be ready for review? Appreciate your time, thank you!

Copy link
Member

@Methuselah96 Methuselah96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, sorry for the delayed review!

@Methuselah96 Methuselah96 enabled auto-merge (squash) January 8, 2023 19:19
@Methuselah96 Methuselah96 merged commit a7729da into reduxjs:main Jan 8, 2023
@smacpherson64
Copy link
Contributor Author

No worries! Thank you!

@smacpherson64 smacpherson64 deleted the adjust-open-location branch January 9, 2023 14:46
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.

2 participants