Skip to content

Allow to use RequestContext extractor in action handlers#11

Merged
aram356 merged 8 commits intomainfrom
feature/action-request-ctx
Nov 7, 2025
Merged

Allow to use RequestContext extractor in action handlers#11
aram356 merged 8 commits intomainfrom
feature/action-request-ctx

Conversation

@aram356
Copy link
Copy Markdown
Contributor

@aram356 aram356 commented Nov 7, 2025

Closes #10

Copy link
Copy Markdown
Contributor

@ChristianPavilonis ChristianPavilonis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but did you mean to do it off of my branch or off of main?

Maybe we should merge mine and rebase or just rebase your commit off of main as it is now.

@aram356
Copy link
Copy Markdown
Contributor Author

aram356 commented Nov 7, 2025

Looks good, but did you mean to do it off of my branch or off of main?

Maybe we should merge mine and rebase or just rebase your commit off of main as it is now.

I branched off your changes for fastly so it should be good to merge this if you ok with it.

@aram356 aram356 merged commit 30f803f into main Nov 7, 2025
2 checks passed
@aram356 aram356 deleted the feature/action-request-ctx branch November 7, 2025 22:35
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.

As developer I want to pass RequestContext to handlers that use #action macro

2 participants