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

[UXE-6102] fix: improve status parsing and handle optional results in list data stream #2091

Merged
merged 4 commits into from
Jan 24, 2025
Merged

[UXE-6102] fix: improve status parsing and handle optional results in list data stream #2091

merged 4 commits into from
Jan 24, 2025

Conversation

lucasmendes21
Copy link
Contributor

@lucasmendes21 lucasmendes21 commented Jan 24, 2025 •
edited
Loading

Bug fix

Explain what was fixed and the correct behavior.

fix: improve status parsing and handle optional results in list data stream

Does this PR introduce UI changes? Add a video or screenshots here.

https://jam.dev/c/8d117706-05f0-418f-b5fb-13ad436cd264

Checklist

Make sure your pull request fits the checklist below (when applicable):

  • The issue title follows the format: [ISSUE_CODE] bug: TITLE
  • Commits are tagged with the right word (feat, test, refactor, etc)
  • Application responsiveness was tested to different screen sizes
  • New tests are added to prevent the same issue from happening again
  • UI changes are validated by a team designer
  • Code is formatted and linted
  • Tags are added to the PR

These changes were tested on the following browsers:

  • Chrome
  • Edge
  • Firefox
  • Safari

@lucasmendes21 lucasmendes21 added the bug Something isn't working label Jan 24, 2025
@lucasmendes21 lucasmendes21 requested review from a team as code owners January 24, 2025 12:34
@lucasmendes21 lucasmendes21 changed the title [UXE-6102] fix: adjust return message 403 [UXE-6102] fix: improve status parsing and handle optional results in list data stream Jan 24, 2025
pauloSF0
pauloSF0 previously approved these changes Jan 24, 2025
View reviewed changes
HerbertJulio
HerbertJulio previously approved these changes Jan 24, 2025
View reviewed changes
@vinigfer vinigfer merged commit 64f1a80 into dev Jan 24, 2025
8 of 12 checks passed
@vinigfer vinigfer deleted the UXE-6102-fix-adjust-return-message-403 branch January 24, 2025 17:45
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@HerbertJulio HerbertJulio HerbertJulio approved these changes

@pauloSF0 pauloSF0 pauloSF0 approved these changes

@peterpaulo-azion peterpaulo-azion peterpaulo-azion left review comments

Assignees

@lucasmendes21 lucasmendes21

Labels
bug Something isn't working
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lucasmendes21 @HerbertJulio @peterpaulo-azion @pauloSF0 @vinigfer