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

- update Locals function to accept interface{} key #2144

Merged
merged 2 commits into from Oct 27, 2022
Merged

- update Locals function to accept interface{} key #2144

merged 2 commits into from Oct 27, 2022

Conversation

pjebs
Copy link
Contributor

@pjebs pjebs commented Oct 7, 2022

See:
#2143
valyala/fasthttp#1387 (merged to master)

@efectn efectn linked an issue Oct 7, 2022 that may be closed by this pull request
3 tasks
@li-jin-gou
Copy link
Contributor

Please follow the PR template @pjebs
https://github.com/gofiber/fiber/blob/master/.github/pull_request_template.md

@ReneWerner87
Copy link
Member

image

i think we have to wait for the fasthttp release

@pjebs
Copy link
Contributor Author

pjebs commented Oct 7, 2022

It'll get tagged on Monday/Tuesday depending on your TZ.

@efectn
Copy link
Member

efectn commented Oct 26, 2022

Can you rebase the changes @pjebs

@pjebs
Copy link
Contributor Author

pjebs commented Oct 26, 2022

@efectn done

@ReneWerner87 ReneWerner87 merged commit ff46029 into gofiber:master Oct 27, 2022
@welcome
Copy link

welcome bot commented Oct 27, 2022

Congrats on merging your first pull request! 🎉 We here at Fiber are proud of you! If you need help or want to chat with us, join us on Discord https://gofiber.io/discord

@pjebs pjebs deleted the Locals-key-interface{} branch October 27, 2022 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🚀 [Feature]: locals accepts interface{} for key
5 participants