Skip to content
This repository has been archived by the owner on Oct 18, 2022. It is now read-only.

Consider moving error-handler into the core framework #18

Open
ben-ryder opened this issue Jun 3, 2022 · 0 comments
Open

Consider moving error-handler into the core framework #18

ben-ryder opened this issue Jun 3, 2022 · 0 comments
Assignees
Labels
improvement Improve an existing feature
Milestone

Comments

@ben-ryder
Copy link
Owner

Package Information: kangojs, error-handler

Describe your improvement
It's likely that the error handler (or a version of it) will be used on every project using KangoJS, so why not just move this functionality to the base framework or at least add some functionality like I have for request auth/validation to make this less custom/external to the framework.

Is your suggestion related to any problems?
Reduces boilerplate and repeated code, error handling will probably rely on dependency injection so it makes sense to possibly absorb the functionality

Additional context
n/a

@ben-ryder ben-ryder added the improvement Improve an existing feature label Jun 3, 2022
@ben-ryder ben-ryder self-assigned this Jun 3, 2022
@ben-ryder ben-ryder added this to the v2 milestone Jun 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
improvement Improve an existing feature
Projects
None yet
Development

No branches or pull requests

1 participant