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

Correct 404 markdown links #2079

Merged
merged 1 commit into from
Dec 4, 2023
Merged

Conversation

matthew-petrie
Copy link
Contributor

These links in the docs were setup incorrectly leading to a 404. This PR corrects the links (removes the preceding /)

@dimsemenov
Copy link
Owner

That's how most links in the documentation are structured, are you deploying the docs in some custom subfolder? Or are you having 404 issues at https://photoswipe.com/ too?

@matthew-petrie
Copy link
Contributor Author

Hm, so interestingly the currently live code works on https://photoswipe.com/methods/ but not on https://github.com/dimsemenov/PhotoSwipe/blob/master/docs/methods.md (other pages links work fine, and I removed the preceding / as is done on other pages - GitHub docs say starting with the / links from the root of the repository

@dimsemenov dimsemenov merged commit ac7eaf6 into dimsemenov:master Dec 4, 2023
@dimsemenov
Copy link
Owner

dimsemenov commented Dec 4, 2023

Ah, I see. I only support docs at https://photoswipe.com/getting-started/ as it includes interactive examples But it wouldn't hurt, thanks!

nvm: had to revert

dimsemenov added a commit that referenced this pull request Dec 4, 2023
dimsemenov added a commit that referenced this pull request Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants