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

Support for Path Parameters? #2

Open
bradcypert opened this issue Jan 28, 2025 · 2 comments
Open

Support for Path Parameters? #2

bradcypert opened this issue Jan 28, 2025 · 2 comments
Labels
accepted enhancement New feature or request

Comments

@bradcypert
Copy link

I'm looking through the documentation for Zinc after having some issues with Zap and I much prefer the API that Zinc offers, however, I'm noticing that we don't really have a way to support path parameters in Zinc. Ideally, I'd be able to do something like the following:

 var z = try zinc.default();
 var router = z.getRouter();

try router.get("/users/:id", getUser);

where id would be parsed out and provided to getUser. I know that this isn't exactly trivial and there's a lot of edge cases here, but I think this would be huge for adoption of Zinc!

Looking forward to hearing back!

@dravenk dravenk added enhancement New feature or request accepted labels Jan 28, 2025
@dravenk
Copy link
Member

dravenk commented Jan 28, 2025

Thank you very much for your feedback. Since I recently joined a new company, I haven't had much time to improve all aspects of the framework. However, I promise that I will definitely add this feature in the future.

@bradcypert
Copy link
Author

Looking forward to it (and possibly interested in helping build it if you'd like, too).

I think you've got a great project and I'd love to see it succeed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants