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

openapi-framework: fix backslahses problems when on windows #873

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rossi-p
Copy link

@rossi-p rossi-p commented May 24, 2023

This commit solves the problem with windows file system when
generating the swagger file with openapi-framework. With this commit
the swagger is now generating the docs with ('/') instead of the ('').

 This commit solves the problem with windows file system when
 generating the swagger file with openapi-framework. With this commit
 the swagger is now generating the docs with ('/') instead of the ('\').
@jsdevel
Copy link
Contributor

jsdevel commented May 25, 2023

@rossi-p how is this different from #869 ?

@rossi-p
Copy link
Author

rossi-p commented May 25, 2023

@jsdevel The PR 869 solves the problem when creating the server with Express, but it still generates a Swagger file with paths using backslashes ( \ ).

@renkei
Copy link

renkei commented Jun 3, 2023

PR 877 is an alternative PR that modifies the package fs-routes instead of openapi-framework. It also solves the swagger issue mentioned by rossi-p.

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.

None yet

3 participants