Skip to content

Conversation

@jeronimoalbi
Copy link
Member

The PR add a new --peer-address to the network chain join command to allow users to skip the interactive "Peer's address" prompt.

@jeronimoalbi
Copy link
Member Author

@lubtd I added the flag during a debugging session and I thought of making a PR for it in case is something we want for the command, otherwise you can close it.

@lumtis
Copy link
Contributor

lumtis commented Oct 14, 2022

@lubtd I added the flag during a debugging session and I thought of making a PR for it in case is something we want for the command, otherwise you can close it.

Thanks for the PR. Yes, this feature is still highly since join could be run on a machine another than the node

Copy link
Contributor

@lumtis lumtis left a comment

Choose a reason for hiding this comment

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

As per Thomas comment, we should not ask for the address in the prompt if provided through the flag

lumtis
lumtis previously approved these changes Oct 17, 2022
tbruyelle
tbruyelle previously approved these changes Oct 17, 2022
@aljo242 aljo242 dismissed stale reviews from tbruyelle and lumtis via a5afa58 October 17, 2022 18:28
@aljo242 aljo242 merged commit 3d9956b into develop Oct 17, 2022
@aljo242 aljo242 deleted the feat/network-chain-join-peer-address branch October 17, 2022 21:08
Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
…te#2921)

* feat: add `--peer-address` flag to `network chain join` command

* chore: change join command to skip prompt when peer address flag is set

Co-authored-by: Alex Johnson <alex@shmeeload.xyz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:network type:new To implement new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants