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.

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

[UXE-6093] fix: adjust origin_id handling in error responses service #2229

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

HerbertJulio
Copy link
Collaborator

@HerbertJulio HerbertJulio commented Feb 28, 2025

Pull Request

What is the new behavior introduced by this PR?

This pull request includes changes to the error response handling in the EdgeApplicationErrorResponsesServices and updates to the test cases and configuration files. The most important changes include modifying the adapt function to handle cases with a single error response, updating test cases to reflect this change, and excluding a specific module from the coverage configuration.

Changes to error response handling:

Updates to test cases:

Does this PR introduce breaking changes?

  • No
  • Yes

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

Does it have a link on Figma?


Checklist

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

  • The issue title follows the format: [ISSUE_CODE] TYPE: TITLE
  • Commits are tagged with the right word (feat, test, refactor, etc)
  • Application responsiveness was tested to different screen sizes
  • Code is formatted and linted
  • Tags are added to the PR

These changes were tested on the following browsers:

  • Chrome
  • Edge
  • Firefox
  • Safari
  • Brave

@HerbertJulio HerbertJulio requested review from a team as code owners February 28, 2025 13:29
@HerbertJulio HerbertJulio changed the title fix: adjust origin_id handling in error responses service [UXE-6093] fix: adjust origin_id handling in error responses service Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant