[API] Turn on/off api endpoints with parameter #12771
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
SirDomin
commented
Jul 2, 2021
•
SirDomin
commented
| Q | A |
|---|---|
| Branch? | 1.8 |
| Bug fix? | no |
| New feature? | yes |
| BC breaks? | no |
| Deprecations? | no |
| Related tickets | |
| License | MIT |
probot-autolabeler
bot
added
the
Documentation
SirDomin
force-pushed
the
api-as-opt
branch
5 times, most recently
from
bcd8cfa
to
1306cf3
Compare
AdamKasp
reviewed
Jul 2, 2021
src/Sylius/Bundle/ApiBundle/ApiPlatform/Bridge/Symfony/Routing/CachedRouteNameResolver.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/test/src/Tests/DisablingApiTest.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Resources/config/services/resolver.xml
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/DependencyInjection/Configuration.php
Outdated
Show resolved
Hide resolved
SirDomin
force-pushed
the
api-as-opt
branch
3 times, most recently
from
fd2397c
to
a6947e4
Compare
GSadee
reviewed
Jul 5, 2021
src/Sylius/Bundle/ApiBundle/EventSubscriber/KernelRequestEventSubscriber.php
Outdated
Show resolved
Hide resolved
probot-autolabeler
bot
added
the
Maintenance
lchrusciel
approved these changes
Jul 5, 2021
src/Sylius/Bundle/ApiBundle/EventSubscriber/KernelRequestEventSubscriber.php
Outdated
Show resolved
Hide resolved
AdamKasp
approved these changes
Jul 6, 2021
src/Sylius/Bundle/ApiBundle/test/src/Tests/DisablingApiTest.php
Outdated
Show resolved
Hide resolved
SirDomin
force-pushed
the
api-as-opt
branch
3 times, most recently
from
681a044
to
022b630
Compare
SirDomin
force-pushed
the
api-as-opt
branch
2 times, most recently
from
63371ad
to
8cb3223
Compare
SirDomin
force-pushed
the
api-as-opt
branch
4 times, most recently
from
78bffb2
to
f454dc7
Compare
|
Thank you, @SirDomin! 🥇 |
|
And this would be me . Just saying thank you I’ve tried a lot of things and just to avail you broke them down. Now what can I do on the owner’s end. I’ve figured out when someone said just go inside and look around. I did in may 2020 life hasn’t been the same, I’m still trying and there still here but I shall conquer this with help from people like you. |
SirDomin
added a commit
that referenced
this pull request
Jul 8, 2021
This PR was merged into the 1.9 branch. Discussion ---------- | Q | A | --------------- | ----- | Branch? | 1.9 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Related tickets | fixes #12771 | License | MIT Commits ------- 91e84e4 [API] Disable API by default 2ecd69b [API] Move note about disabled API to proper section
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.