Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

require the at option when scheduling backups #1485

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

neovintage
Copy link
Contributor

Small UX problem whereby someone creates a schedule and then issues the same command without the at flag thinking they could view schedules. The schedule is overwritten and set back to 4:00 UTC.

/cc @uhoh-itsmaciek @chadbailey59

What do you guys think?

@msakrejda
Copy link
Contributor

Interesting idea. It seems weird to have required flags (options are usually... optional), but it might make sense here. Are there other instances of this behavior in the CLI?

@jdx
Copy link
Contributor

jdx commented Apr 3, 2015

perhaps prompt if no time is scheduled instead of forcing them to run the command again?

@neovintage
Copy link
Contributor Author

@uhoh-itsmaciek I quickly grep-ed across codebase and didn't see it being done anywhere else. I wouldn't want to be the first one to start requiring options.

@dickeyxxx That's a good idea.

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

Successfully merging this pull request may close these issues.

3 participants