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

Backend Error Handling #131

Closed
gr812b opened this issue Oct 14, 2024 · 1 comment · Fixed by #149
Closed

Backend Error Handling #131

gr812b opened this issue Oct 14, 2024 · 1 comment · Fixed by #149
Assignees
@hydrowoxy
Labels
backend Involves the Java Quarkus backend code quality Improve the current code enhancement New feature or request
Milestone
Data Viewer 1.2

Comments

@gr812b
Copy link
Collaborator

gr812b commented Oct 14, 2024

Despite the recent backend improvements, the errors are still all over the place. Catching all default errors and throwing custom ones in their place (and creating mappers so the front end gets a descriptive error message) needs to be done across the entire java codebase.
@gr812b gr812b added enhancement New feature or request code quality Improve the current code labels Oct 14, 2024
@gr812b gr812b added this to the Data Viewer 1.2 milestone Oct 14, 2024
@gr812b
Copy link
Collaborator Author

gr812b commented Nov 13, 2024

Discussion on utilizing AOP (Aspect-Oriented Programming) to handle errors in a centralized manner.

@gr812b gr812b added the backend Involves the Java Quarkus backend label Nov 13, 2024
@gr812b gr812b linked a pull request Dec 15, 2024 that will close this issue
Error Handling with AOP #149
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hydrowoxy hydrowoxy

Labels
backend Involves the Java Quarkus backend code quality Improve the current code enhancement New feature or request
Projects
None yet
Milestone
Data Viewer 1.2
Development

Successfully merging a pull request may close this issue.

Error Handling with AOP McMaster-Baja-Racing/Better-Data-Viewer
2 participants
@hydrowoxy @gr812b