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

config command lacks explanations on how to manage arrays #10645

Closed
2 tasks done
naebushi opened this issue Dec 27, 2024 · 0 comments · Fixed by #10650
Closed
2 tasks done

config command lacks explanations on how to manage arrays #10645

naebushi opened this issue Dec 27, 2024 · 0 comments · Fixed by #10650
Assignees
Labels
P1 High: Likely tackled by core team if no one steps up topic/docs-ipfs Topic docs-ipfs

Comments

@naebushi
Copy link

Checklist

Location

var ConfigCmd = &cmds.Command{

Description

It is impossible to determine how to edit config params containing arrays. For example, none of these seemingly intuitive commands work:

ipfs config --json Addresses.AppendAnnounce '{0: "/ip4/some_ip/tcp/4001"}'`
ipfs config Addresses.AppendAnnounce ["/ip4/some_ip/tcp/4001"]
ipfs config --json Addresses.AppendAnnounce ["\/ip4\/some_ip\/tcp\/4001"]
ipfs config Addresses.AppendAnnounce ["/ip4/some_ip/tcp/4001"]
ipfs config Addresses.AppendAnnounce "/ip4/some_ip/tcp/4001"
ipfs config Addresses.AppendAnnounce[] "/ip4/some_ip/tcp/4001"

I suggest adding some clarifiation regarding array management

@naebushi naebushi added need/triage Needs initial labeling and prioritization topic/docs-ipfs Topic docs-ipfs labels Dec 27, 2024
@gammazero gammazero added P1 High: Likely tackled by core team if no one steps up and removed need/triage Needs initial labeling and prioritization labels Jan 2, 2025
@gammazero gammazero self-assigned this Jan 2, 2025
lidel pushed a commit that referenced this issue Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 High: Likely tackled by core team if no one steps up topic/docs-ipfs Topic docs-ipfs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants