Skip to content

Add TransportError to CableUxUpdate#129

Merged
AlfioEmanueleFresta merged 1 commit into
masterfrom
ux-errors-2
Jul 21, 2025
Merged

Add TransportError to CableUxUpdate#129
AlfioEmanueleFresta merged 1 commit into
masterfrom
ux-errors-2

Conversation

@AlfioEmanueleFresta

Copy link
Copy Markdown
Member

Requires #128.

  • Surfaces asynchronous transport errors via CableUxUpdate::Error.
  • All Cable errors to use TransportError, rather than the generic Error.

@msirringhaus msirringhaus left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AlfioEmanueleFresta

Copy link
Copy Markdown
Member Author

Rebased

Base automatically changed from ux-errors-1 to master July 21, 2025 14:42
@AlfioEmanueleFresta AlfioEmanueleFresta merged commit b411964 into master Jul 21, 2025
4 checks passed
@AlfioEmanueleFresta AlfioEmanueleFresta deleted the ux-errors-2 branch July 21, 2025 14:56
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.

2 participants