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

[Documentation] Sylius Plus docs part 2; including fixes to links #11177

Merged
merged 5 commits into from
Mar 5, 2020

Conversation

CoderMaggie
Copy link
Member

Q A
Branch? 1.6
Doc fix? yes - broken links for The Book
New docs? yes
Related tickets Part 2 of #11054
License MIT

@CoderMaggie CoderMaggie requested a review from a team as a code owner March 4, 2020 13:31
docs/book/architecture/emails.rst Outdated Show resolved Hide resolved
docs/book/architecture/emails.rst Outdated Show resolved Hide resolved
docs/book/architecture/emails.rst Show resolved Hide resolved
* :doc:`/book/customers/customer_and_shopuser`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* :doc:`/book/customers/customer_and_shopuser`
* :doc:`/book/customers/customer_and_shop_user`

? 😄

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the file is named like that already, can be changed but ina different PR.

* :doc:`/book/customers/customer_and_shopuser`
* :doc:`/book/customers/customer_pools`
* :doc:`/book/customers/adminuser`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* :doc:`/book/customers/adminuser`
* :doc:`/book/customers/admin_user`

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, it's the filename, I cannot change it in the scope of this PR.

<li><p><a class="reference internal" href="orders/refunds.html"><span class="doc">Refunds</span></a></p></li>
</ul>
* :doc:`/book/orders/orders`
* :doc:`/book/orders/cart-flow`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So what convention do we have? - or _?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

_ in general, but there are some mistakes in the filenames made previously.

@GSadee GSadee added the Documentation Documentation related issues and PRs - requests, fixes, proposals. label Mar 5, 2020
@GSadee GSadee merged commit 1969928 into Sylius:1.6 Mar 5, 2020
@GSadee
Copy link
Member

GSadee commented Mar 5, 2020

Thanks, Magdalena! 🥇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Documentation related issues and PRs - requests, fixes, proposals.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants