Conversation
|
@pat-s review when you can! One point of friction right now with the cli is that we cannot update settings from it yet. The idea here is that we can fetch the current remote settings as a |
pat-s
left a comment
There was a problem hiding this comment.
This "sounds" wrong to me reading-wise, i.e. item as the subcommand and then a format as the key for it.
Something like the following feels more "common":
ricochet content <id> --configricochet app <id> --config
Rationale: without --config, an overview of the content/app is returned. --config returns specifically` the TOML config. It's always TOML, so no need to specify that one explicitly.
The API endpoint is also named content, so sticking to similar naming as we already have likely makes sense overall.
|
Heard. My thinking is that we should be able to collect content item commands under a subcommand like:
Then we would be able to have other grouped API endpoints as subcommands like |
fcd2e44 to
57917f8
Compare
|
I don't think we can merge apps and tasks under one command as not all actions/subcommands would apply to both. How about just reuse the existing terminology?
This would also align with the current API endpoint structure. There is no '/tasks/` yet but we could add this group? |
This PR adds a
ricochet itemsubcommand with the first command beingtoml.