-
Notifications
You must be signed in to change notification settings - Fork 4
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
Frontend error handling #495
base: main
Are you sure you want to change the base?
Conversation
…question into frontend_error_handling
@@ -1,13 +1,15 @@ | |||
import api from "@/utils/api"; | |||
import api, { CustomError, handleApiError } from "@/utils/api"; | |||
import axios from "axios"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unused import
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed
{snackbarMessage.message} | ||
</Alert> | ||
</Snackbar> | ||
</Layout.FlexBox> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is an tag that closed here but that was never opened. Probably happened during the merge with main
Thanks @WanjohiWanjohi, There i an issue with the urgency detection page whenever I click on it I get this : |
Reviewer:@lickem22
Estimate: 30 minutes
Ticket
Fixes: AAQ-897
Description
Goal
The goal of this PR is to improve error handling to show consistent errors using a SnackBar or Alert
Changes
The following changes have been made:
CustomError
utility in theintegrations
andurgency_rules
componentsHow has this been tested?
Tested locally.
Checklist
Fill with
x
for completed.