Skip to content

Report rest client errors, dispose coroutine scopes & disable last Next button #23

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

Merged
merged 3 commits into from
Jun 24, 2022

Conversation

fioan89
Copy link
Collaborator

@fioan89 fioan89 commented Jun 24, 2022

  • Catch&report errors raised by the REST client

  • the exceptions from the rest client were handled by
    the coroutine scope which canceled everything in the
    scope without any event logged to the user.

  • the exceptions are now catched inside the coroutine scope
    & logged.

  • any error logged through the diagnostic logger is also displayed
    in the Gateway UI through the internal errors message panel (red
    balloon in the bottom right corner)

  • Dispose the coroutine scope when is no longer needed

  • disable next action button in the last step of the wizard when no IDE could be resolved

fioan89 added 3 commits June 25, 2022 00:18
- the exceptions from the rest client were handled by
  the coroutine scope which canceled everything in the
  scope without any event logged to the user.
- the exceptions are now catched inside the coroutine scope
  & logged.
- any error logged through the diagnostic logger is also displayed
  in the Gateway UI through the internal errors message panel (red
  balloon in the bottom right corner)
@fioan89 fioan89 merged commit 41728d9 into main Jun 24, 2022
@fioan89 fioan89 deleted the report-rest-client-errors branch November 28, 2022 20:32
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.

1 participant