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

Implement OperationHandler for axum::handler::Layered #133

Merged
merged 1 commit into from
May 15, 2024

Conversation

svix-jplatte
Copy link
Contributor

@svix-jplatte svix-jplatte commented Apr 19, 2024

Allow adding handlers to ApiMethodRouter that have a layer applied via Handler::layer.

@svix-jplatte
Copy link
Contributor Author

Ping @Wicpar

@Wicpar Wicpar merged commit 2bad9ab into tamasfe:master May 15, 2024
1 check passed
@Wicpar
Copy link
Collaborator

Wicpar commented May 15, 2024

remind me to release if i haven't done it by tomorrow

@svix-jplatte svix-jplatte deleted the layered-handler branch May 15, 2024 10:59
@svix-jplatte
Copy link
Contributor Author

svix-jplatte commented May 15, 2024

Actually, I don't really care about this going into aide 0.13 for now, as we're still using 0.12. So if you could make a v0.12.x branch that I can make a backport PR to, that would actually be more helpful for me ^^

See also #132

And thanks for the fast response after pinging :)

@svix-jplatte
Copy link
Contributor Author

@Wicpar Reminder to follow up

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

Successfully merging this pull request may close these issues.

2 participants