You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
heroku certs:delete --app mathdown
› Warning: certs:delete is not a heroku command.
Did you mean apps:delete? [y/n]: y
^C
Pressing "y" there was silly on my part (and good thing I aborted in time 😓) but IMHO apps:delete action is to dangerous to be suggested for ANY other command, and certainly for commands that deal with another noun (such as certs: here). 🛑
What is the expected behavior?
The suggestion I really needed was certs:remove.
One approach is give all "remove" commands a "delete" alias consistently, a-la commit 6b41d18.
But I feel discoverability is secondary goal here, it's more important to add some defense from undesired destructive action. Please consider:
excluding destructive actions from suggestions? This may be too strong, e.g. for app:destry the intent is clearly apps:destroy.
listing destructive actions as textual suggestions, but not offering direct [y/n] activation?
offering but with default [y/N]?
(I'm not sure if that's already the default, lowercase y/n give no hint; capital N could give subtle hint you may want to stop and think — but OTOH it's well known that users get trained confirm such prompts without thinking...)
avoid suggesting same verb for different noun? In general these are not likely to match user's intent... Well maybe if the noun is light mispelling e.g. appps:destroy, but not for existing different noun certs:.
heroku/7.60.2 linux-x64 node-v16.14.0 on Fedora 36 linux.
The text was updated successfully, but these errors were encountered:
Do you want to request a feature or report a bug?
usability bug.
What is the current behavior?
Pressing "y" there was silly on my part (and good thing I aborted in time 😓) but IMHO
apps:delete
action is to dangerous to be suggested for ANY other command, and certainly for commands that deal with another noun (such ascerts:
here). 🛑What is the expected behavior?
The suggestion I really needed was
certs:remove
.One approach is give all "remove" commands a "delete" alias consistently, a-la commit 6b41d18.
But I feel discoverability is secondary goal here, it's more important to add some defense from undesired destructive action. Please consider:
app:destry
the intent is clearlyapps:destroy
.(I'm not sure if that's already the default, lowercase y/n give no hint; capital N could give subtle hint you may want to stop and think — but OTOH it's well known that users get trained confirm such prompts without thinking...)
appps:destroy
, but not for existing different nouncerts:
.heroku/7.60.2 linux-x64 node-v16.14.0
on Fedora 36 linux.The text was updated successfully, but these errors were encountered: