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

feat: add error handling #514

Merged
merged 2 commits into from
Dec 23, 2024
Merged

feat: add error handling #514

merged 2 commits into from
Dec 23, 2024

Conversation

d-koppenhagen
Copy link
Member

@d-koppenhagen d-koppenhagen commented Dec 21, 2024

Closes #352
Based on #512 and #513 which must be merged first

@d-koppenhagen d-koppenhagen force-pushed the validate-format branch 3 times, most recently from 43681e3 to 65d1f5d Compare December 22, 2024 12:11
@d-koppenhagen d-koppenhagen requested a review from Morl99 December 22, 2024 12:12
@d-koppenhagen d-koppenhagen force-pushed the validate-format branch 2 times, most recently from 04ed0c8 to c8a3fb3 Compare December 22, 2024 19:46
@d-koppenhagen d-koppenhagen marked this pull request as ready for review December 22, 2024 19:46
.github/workflows/build.yml Outdated Show resolved Hide resolved
@d-koppenhagen
Copy link
Member Author

@Morl99 I needed to add a fix for #522 as well as a test since it was damaged by the vue3/env update. Please simply rebase if it fits (commits should be kept so don't squash)

@d-koppenhagen
Copy link
Member Author

@Morl99 I just notices that currently using pipelines will fail to open the report since the url param was not handled. I will merge this PR to fix the bug and also improve error handling. Feel free to comment anyway, Or raise an issue for any findings afterwards.

@d-koppenhagen d-koppenhagen merged commit 06d63e0 into main Dec 23, 2024
1 check passed
Copy link

🎉 This PR is included in version 1.5.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error handling while loading a new report
2 participants