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

Use API Url prefix on sylius.api.paths_to_hide #13901

Merged
merged 1 commit into from
Apr 29, 2022

Conversation

Prometee
Copy link
Contributor

Q A
Branch? master
Bug fix? yes (in case you are not using the default api url prefix)
New feature? no
BC breaks? no
Deprecations? no
Related tickets none
License MIT

If you are using another url prefix, different from /api/v2 then those paths are not processed, this PR fix this issue.

@Prometee Prometee requested a review from a team as a code owner April 27, 2022 08:44
@probot-autolabeler probot-autolabeler bot added the API APIs related issues and PRs. label Apr 27, 2022
@Prometee Prometee force-pushed the api-url-prefix-paths-to-hide branch from 6f0eefc to 6f15f12 Compare April 27, 2022 08:46
@lchrusciel lchrusciel added the Bug Confirmed bugs or bugfixes. label Apr 29, 2022
@lchrusciel lchrusciel merged commit 50be917 into Sylius:master Apr 29, 2022
@lchrusciel
Copy link
Member

Thanks, Francis! 🎉

@Prometee Prometee deleted the api-url-prefix-paths-to-hide branch April 30, 2022 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API APIs related issues and PRs. Bug Confirmed bugs or bugfixes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants