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

Please make "fn delete ..." error messages more user-friendly #394

Open
sachin-pikle opened this issue Aug 26, 2018 · 2 comments
Open

Please make "fn delete ..." error messages more user-friendly #394

sachin-pikle opened this issue Aug 26, 2018 · 2 comments

Comments

@sachin-pikle
Copy link

sachin-pikle commented Aug 26, 2018

In both cases below I get an error, which is expected. But the message is cryptic and needs to be made more user-friendly.

  1. Trying to delete an app with a function and a trigger.

$ fn delete app test-app
Fn: [DELETE /apps/{app}][409] DeleteAppsApp default &{Error:0xc420449440}
See 'fn --help' for more information. Client version: 0.4.150

  1. Trying to delete a function with a trigger.

$ fn delete fn lisbon-app node-t-fn
Fn: [DELETE /fns/{fnID}][409] DeleteFn default &{Fields: Message:Fn has associated Triggers}
See 'fn --help' for more information. Client version: 0.4.150

@sachin-pikle sachin-pikle changed the title Make error message more user-friendly Please make error message more user-friendly Aug 26, 2018
@sachin-pikle sachin-pikle changed the title Please make error message more user-friendly Please make "fn delete ..." error messages more user-friendly Aug 26, 2018
@rdallman
Copy link
Contributor

this is from swagger... attempted to do this in the last cli but it's kinda shit, have to type switch all the errors iirc. but yea, agree, we need to unroll the errors

@rikgibson
Copy link
Contributor

This ties into #333 I think.

@rikgibson rikgibson added this to the M2 milestone Sep 10, 2018
@rikgibson rikgibson removed their assignment Oct 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants