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

Restructure filemanager commands #1258

Open
wants to merge 2 commits into
base: next
Choose a base branch
from

Conversation

brandenrodgers
Copy link
Contributor

@brandenrodgers brandenrodgers commented Nov 22, 2024

Description and Context

Refactoring the filemanager commands. This one is pretty small. Just removing the options that show when running hs filemanager with no sub command.

I also found a bug in the fetch command where we weren't respecting the override option. I don't think it was being used anywhere in the upload command so I didn't add it in there.

hs filemanager

  • Update description to: "Commands for managing files in the File Manager."
  • Remove all flags except for the --help flag because they don't do anything
  • Fix duplicate argument issue for config and account in nested commands

hs filemanager upload

  • Update description to: "Upload a folder or file to the File Manager."
  • Remove --version flag because it's unnecessary

hs filemanager fetch

  • Update description to: "Fetch a folder or file from the File Manager."
  • Remove --version flag because it's unnecessary

Screenshots

TODO

Who to Notify

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.

1 participant