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.

Sign up for GitHub

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

Jump to bottom

Handle Addin Errors in UI #35

Open
jsdbroughton opened this issue Jun 13, 2022 · 0 comments
Open

Handle Addin Errors in UI #35

jsdbroughton opened this issue Jun 13, 2022 · 0 comments

Comments

@jsdbroughton
Copy link
Member

Errors in the commit elements adddin method have started to handle SpeckleExceptions and NotifyUI of the error rather than throwing the error unhandled.

For now, non SpeckleExceptions are rethrown.

This is embryonic Exception handling and will need to manage the full set of potential errors, for now these are the ones reported to the UI.

UI currently has no response to the Error and the commit selection option remains disabled.
jsdbroughton added a commit that referenced this issue Jun 13, 2022
@jsdbroughton
fix (addin): Begun Exception handling for Speckle server errors.
bd3224a 
TODO: Reacting to the handled errors needs UI handling (#35)
@jsdbroughton jsdbroughton mentioned this issue Jun 13, 2022
Merged
jsdbroughton added a commit that referenced this issue Jun 13, 2022
@jsdbroughton
fix (addin): Begun Exception handling for Speckle server errors.
e80c73e 
TODO: Reacting to the handled errors needs UI handling (#35)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Rimshot
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jsdbroughton