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

[DOCS] Fix relative paths #606

Merged
merged 2 commits into from
Jan 6, 2024
Merged

Conversation

FBruzzesi
Copy link
Collaborator

@FBruzzesi FBruzzesi commented Dec 31, 2023

Description

In the deployed docs in github pages, all relative links are broken. I would imagine that the issue is similar to what happened in #602 .

This PR tries to patch that.

Most annoying fact is that locally both strategies work.

Edit: Adding screenshot as example:

Clicking the link at https://koaning.github.io/scikit-lego/#usage to open the user guide

image

maps to https://koaning.github.io/user-guide/datasets (which is 404) instead of https://koaning.github.io/scikit-lego/user-guide/datasets/

@FBruzzesi FBruzzesi requested a review from koaning January 3, 2024 13:16
@FBruzzesi FBruzzesi added documentation We might describe something better bug Something isn't working labels Jan 3, 2024
@@ -1,16 +1,16 @@
[![Build status](https://github.com/koaning/scikit-lego/workflows/Unit%20Tests/badge.svg)](https://github.com/{github_id}/{repository}/workflows/{workflow_name}/badge.svg)
Copy link
Collaborator Author

@FBruzzesi FBruzzesi Jan 4, 2024

Choose a reason for hiding this comment

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

Not sure if Build status is still relevant, I couldn't find a passing status though.
The adjusted link leads to "no status"

@koaning koaning merged commit bde1c6e into koaning:main Jan 6, 2024
14 checks passed
@FBruzzesi FBruzzesi deleted the docs/broken-paths branch January 8, 2024 08:06
@FBruzzesi FBruzzesi restored the docs/broken-paths branch January 19, 2024 18:36
@FBruzzesi FBruzzesi deleted the docs/broken-paths branch January 19, 2024 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation We might describe something better
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants