Skip to content
This repository has been archived by the owner on Apr 19, 2024. It is now read-only.

Fix internal docs links #4

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

Conversation

vfonic
Copy link
Contributor

@vfonic vfonic commented Aug 17, 2021

I noticed that the links I modified yesterday got broken because I didn't link to .md, but left the links with no file extension.

For example:

- [Zone record](shipments#zones)
+ [Zone record](shipments.md#zones)

Seems like GitBook changes
[Zone record](shipments#zones)
to
[Zone record](https://github.com/spree/spree-dev-docs/tree/0628094f68853238d9b13aa3b24d7b1e1b13fca4/internals/shipments/README.md#zones)
(a commit pointing to README, but not even that as it points to README in internals/shipments)

This link seems to be converted correctly, but please check:
[Zone record](shipments.md#zones) => [Zone record](/internals/shipments#zones)

I found out that I need to add .md file extension from links on other docs pages that seem to work. For example:
https://github.com/spree/spree-dev-docs/edit/master/internals/products.md

@vfonic
Copy link
Contributor Author

vfonic commented Aug 18, 2021

This has been rebased on top of master now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant