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

formatParamUrl should be exposed #747

Closed
2 tasks done
marcopiraccini opened this issue Aug 17, 2023 · 5 comments · Fixed by #748
Closed
2 tasks done

formatParamUrl should be exposed #747

marcopiraccini opened this issue Aug 17, 2023 · 5 comments · Fixed by #748

Comments

@marcopiraccini
Copy link

Prerequisites

  • I have written a descriptive issue title
  • I have searched existing issues to ensure the feature has not already been requested

🚀 Feature Proposal

The formatParamUrl function HERE translates path in the fastify router format to openapi.
It would be great to have it exposed as decorator, so it could be used from the same purpose on other projects.

Motivation

I need to do exactly what this function does, but is not exposed from this module.

Example

No response

@Uzlopak
Copy link
Contributor

Uzlopak commented Aug 17, 2023

Should it really be a decorator? And if you how name it?
Cant we just expose it to be requirable?

@mcollina
Copy link
Member

We can also expose it to be requirable, it would work well for us too.

@Uzlopak
Copy link
Contributor

Uzlopak commented Aug 17, 2023

I think exposing would be cleaner.

@Uzlopak
Copy link
Contributor

Uzlopak commented Aug 17, 2023

see #748

@marcopiraccini
Copy link
Author

Yes, requirable is OK.

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 a pull request may close this issue.

3 participants