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
[Docs] Add and improve new api docs #12599
Conversation
AdamKasp
commented
May 5, 2021
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
License | MIT |
docs/customization/api.rst
Outdated
================================== | ||
|
||
Let's assume that your shop offer only digital products so you don't need setting shipment method at all. | ||
To remove endpoint you only need to delete unnecessary configuration from ``config/api_platform/Order.xml`` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this sentence true? I mean we have some underlying logic that changes states of cart and checkout. Speaking of endpoint only POV, well we can just remove the endpoint and go drink some margarita 🍸 , but I don't think we can finish checkout process 🙁
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, but actually no :D
if you have only one shipping method and enabled option Skip shipping step if only one shipping method is available?
on the channel, while checkout you don't need to set shipping method :)
you can check how it works on UI :)
05db891
to
95b0f0a
Compare
95b0f0a
to
004f4e1
Compare
Thanks, Adam! 🥇 |
# config/packages/api_platform.yaml | ||
api_platform: | ||
mapping: | ||
paths: | ||
- '%kernel.project_dir%/vendor/sylius/sylius/src/Sylius/Bundle/ApiBundle/Resources/config/api_resources' | ||
- '%kernel.project_dir%/config/api_platform' | ||
patch_formats: | ||
json: ['application/merge-patch+json'] | ||
swagger: | ||
versions: [3] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we provide this config in Sylius Standard?