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

Switched from routes to self-registering handlers. #59

Open

mikestefanello wants to merge 2 commits into main from handlers-2

Owner

mikestefanello commented Mar 30, 2024 •

edited

Improvements:

Use handlers that group related routes rather than individual routes with methods for each HTTP method.
Allow handlers to self-register middleware, groups and routes rather than having a single method construct every single route.
Inject the container via a method rather than initializing each handler individually.
Use init() in each handler file to add itself to the list of handlers to add to the router (not sure about this)

Still need to update the README.

mikestefanello added 2 commits

March 30, 2024 09:56


          Switched from routes to self-registering handlers.

bf7722b


          Redefine handler interface.

492228f

mikestefanello mentioned this pull request

Thoughts on Example for Splitting Out "Business Logic Layer"? #63

Closed

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