unison icon indicating copy to clipboard operation
unison copied to clipboard

UX Proposal: Remove `push.create` in favor of `push` with a confirmation

Open hojberg opened this issue 3 years ago • 1 comments

It's very easy to use push when pushing a new path to Share, resulting in an error message suggesting to use push.create instead.

We have this approach to help avoid users making mistakes, typo-ing and accidentally pushing to a location they didn't intend.

I think it'd help with ergonomics and paving the road most threaded by prompting the user with a Y/n question when encountering a new push destination and removing push.create entirely.

Suggested message:

It looks like `hojberg.public.textExtra.v2` doesn't exist in Unison Share.
Do you want to create it and push there now? Y/n

Open to alternative approaches that help mitigate this papercut that most users run into.


Also see prompts ticket: https://github.com/unisonweb/unison/issues/3422

There's a few other tickets related to the push command that might be worth taking into account/consolidating with this effort:

  • https://github.com/unisonweb/unison/issues/3270
  • https://github.com/unisonweb/unison/issues/3084

hojberg avatar Sep 15 '22 20:09 hojberg

I totally agree.

Once we properly support a non-interactive mode (for example when not in a tty), we won't want to prompt in that one. I'm not sure whether the right answer in that scenario is to fail or go ahead and push.

ceedubs avatar Sep 16 '22 12:09 ceedubs