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

Enable error management on page running a process #1154

Open
wants to merge 15 commits into
base: dev
Choose a base branch
from
Open

Enable error management on page running a process #1154

wants to merge 15 commits into from

Conversation

Quetzacoalt91
Copy link
Member

Questions Answers
Description? Following #1135, the error management is now enabled on the process page as well.
Type? new feature
BC breaks? Nope
Deprecations? Nope
Fixed ticket? /
Sponsor company @PrestaShopCorp
How to test? Failed HTTP requests are now visible when happening during a process, and show the page in an errored state. Sending the error report will attach the details shown and the errored HTTP request as an attachment.

@Quetzacoalt91 Quetzacoalt91 added the Blocked Status: The issue is blocked by another task label Feb 4, 2025
@Quetzacoalt91 Quetzacoalt91 self-assigned this Feb 4, 2025
@Quetzacoalt91
Copy link
Member Author

Requires #1135 to be merged first.

@Quetzacoalt91 Quetzacoalt91 added the new feature Type: New feature label Feb 6, 2025
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Feb 7, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
3.8% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Quetzacoalt91 Quetzacoalt91

Labels
Blocked Status: The issue is blocked by another task new feature Type: New feature
Projects
PR Dashboard
Status: Ready for review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Quetzacoalt91 @tblivet