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

add error code #14

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

add error code #14

wants to merge 4 commits into from

Conversation

kktk-KO
Copy link

@kktk-KO kktk-KO commented Jun 22, 2017

No description provided.

@liuhuanjim013
Copy link
Member

error message is supposed to provide details. what's the new use case you are trying to cover?

@rdiankov
Copy link
Member

i think it is fine. please comment on detailerrorcode and rename detail_error_code

thanks

@kktk-KO
Copy link
Author

kktk-KO commented Jun 22, 2017

currently the exception contains locale-specific message which makes it hard to handle errors.
and existing error code is the name of high-level error type like BackendSlaveError which is too obscure.

@kktk-KO
Copy link
Author

kktk-KO commented Jun 22, 2017

I am now adding KinBodyNotFoundByName error code to planningcommon for example to handle this in binpickingsystemanager

kanbouchou pushed a commit that referenced this pull request Nov 28, 2022
WIP: Add command getting detection history
felixvd pushed a commit that referenced this pull request Mar 21, 2023
change params of UpdateObjects
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.

3 participants