Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Added displaying error messages in UI#87

Merged
andrewpareles merged 9 commits intovoideditor:mainfrom
w1gs:main
Oct 18, 2024
Merged

Added displaying error messages in UI#87
andrewpareles merged 9 commits intovoideditor:mainfrom
w1gs:main

Conversation

@w1gs
Copy link
Contributor

@w1gs w1gs commented Oct 4, 2024

This PR adds error messages in the extension UI. This should make it easier for new users to understand why a request has failed. This should also handle aborting correctly so if there are any errors, the UI features that depend on aborted connections (like resetting the state of the send button from the 'stop' state) also get updated.

Examples:
CleanShot 2024-10-03 at 21 36 40@2x

CleanShot 2024-10-03 at 21 37 19@2x

CleanShot 2024-10-03 at 21 38 31@2x

CleanShot 2024-10-03 at 21 39 56@2x

CleanShot 2024-10-03 at 21 42 35@2x

@andrewpareles
Copy link
Contributor

What autoformatter are you using?

@w1gs
Copy link
Contributor Author

w1gs commented Oct 7, 2024

What autoformatter are you using?

I used prettier with the configuration mentioned in #72. I could revert it if needed.

@andrewpareles andrewpareles added the high priority High priority issue label Oct 15, 2024
@andrewpareles
Copy link
Contributor

andrewpareles commented Oct 17, 2024

Will merge this shortly!

andrewpareles added a commit that referenced this pull request Oct 17, 2024
@andrewpareles andrewpareles changed the base branch from main to ui-refactor-10-15 October 17, 2024 23:47
@andrewpareles andrewpareles changed the base branch from ui-refactor-10-15 to main October 18, 2024 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

high priority High priority issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants