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 ./manage cli documentation or functionality #269

Open
WadeBarnes opened this issue Jun 22, 2023 · 4 comments
Open

Update ./manage cli documentation or functionality #269

WadeBarnes opened this issue Jun 22, 2023 · 4 comments

Comments

@WadeBarnes
Copy link
Member

The ./manage cli command no longer works as described in the Connecting to the Network section of the main readme.

Either update the documentation to reference the Using the containerized indy-cli, or modify the scripts so the command functions as described.

@WadeBarnes
Copy link
Member Author

@swcurran, @esune, Any opinion on which option you'd like to see implemented?

@swcurran
Copy link
Contributor

Given that ./manage is ubiquitous in von-network, my gut says to fix the scripts to make sense, and adjust the documentation if there is a gap.

If the other reference is complete and sufficient, then removing the “cli” section from ./manage would be the next minimum.

@WadeBarnes:

  • Which is the least estimated effort, and by how much?
  • Which do you think is the best result?

@WadeBarnes
Copy link
Member Author

It would be nice to have a simplified startup process that works with any ledger, not just von-network. The previous functionality described by the documentation relied on the IP address to point to a von-network instance where it could infer the URL to the genesis file. So after thinking about this for a bit, it might not be appropriate to resurrect the documented functionality.

Updating the documentation would be the least amount of effort. Resurrecting the documented functionality would take an hour or so.

@esune
Copy link
Member

esune commented Jul 7, 2023

If I got it right, I agree that updating the docs to use the newer/up-to-date method (containerized cli) is probably the best way forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants