Skip to content

fix: Fix API error with stream #459

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
Merged
merged 1 commit into from
Aug 8, 2025
Merged

fix: Fix API error with stream #459

merged 1 commit into from
Aug 8, 2025

Conversation

Pijukatel
Copy link
Contributor

  • Handle properly API error with stream
  • Add tests

@Pijukatel Pijukatel added bug Something isn't working. t-tooling Issues with this label are in the ownership of the tooling team. labels Aug 8, 2025
@github-actions github-actions bot added this to the 120th sprint - Tooling team milestone Aug 8, 2025
@github-actions github-actions bot added the tested Temporary label used only programatically for some analytics. label Aug 8, 2025
@Pijukatel Pijukatel added the adhoc Ad-hoc unplanned task added during the sprint. label Aug 8, 2025
@Pijukatel Pijukatel requested a review from Mantisus August 8, 2025 12:53
Mantisus
Mantisus approved these changes Aug 8, 2025
View reviewed changes
Copy link
Collaborator

@Mantisus Mantisus left a comment

Choose a reason for hiding this comment

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

LGTM

@Pijukatel Pijukatel marked this pull request as ready for review August 8, 2025 13:04
@Pijukatel Pijukatel requested a review from MQ37 August 8, 2025 13:06
@Pijukatel Pijukatel merged commit 0c91ca5 into master Aug 8, 2025
31 of 32 checks passed
@Pijukatel Pijukatel deleted the fix-api-error-in-stream branch August 8, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MQ37 MQ37 MQ37 approved these changes

+1 more reviewer

@Mantisus Mantisus Mantisus approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@Pijukatel Pijukatel

Labels
adhoc Ad-hoc unplanned task added during the sprint. bug Something isn't working. t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.
Projects
None yet
Milestone
120th sprint - Tooling team
Development

Successfully merging this pull request may close these issues.
3 participants
@Pijukatel @MQ37 @Mantisus