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

503 fix broken links #504

Merged
merged 5 commits into from
Jun 29, 2023
Merged

503 fix broken links #504

merged 5 commits into from
Jun 29, 2023

Conversation

younesStrittmatter
Copy link
Collaborator

Description

Type of change

Delete as appropriate:

  • docs: Documentation only changes

Questions (Optional)

the links in the jupiter notebooks work, but it doesn't seem right. I brievly looked into the mkdocs-jupiter plugin documentation but didn't find much. I'll work on it when I find time, but this is a quick fix so at least it works.

@younesStrittmatter younesStrittmatter linked an issue Jun 29, 2023 that may be closed by this pull request
Copy link
Collaborator

@benwandrew benwandrew left a comment

Choose a reason for hiding this comment

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

so the links as you changed them were not working for me, so i tested reverting one of the paths to the original and now it works. not sure why they weren't working for you before, or why they are working for me now...

@younesStrittmatter
Copy link
Collaborator Author

Weird... but they are not working on the website as they are now (../experimentalis/index.md) See here: https://autoresearch.github.io/autora/tutorials/

@younesStrittmatter
Copy link
Collaborator Author

Or are the website links working for you?

@younesStrittmatter
Copy link
Collaborator Author

online I mean. Not locally.

@benwandrew
Copy link
Collaborator

ah no, you're right. it's not working for me online as it originally was. happy to undo my commit and approve then as a temporary fix (at least to see what happens after merging this PR and rebuilding the site).

@benwandrew benwandrew self-requested a review June 29, 2023 23:20
@younesStrittmatter younesStrittmatter added this pull request to the merge queue Jun 29, 2023
Merged via the queue into main with commit c8b033f Jun 29, 2023
@younesStrittmatter younesStrittmatter deleted the 503-fix-broken-links branch June 29, 2023 23:31
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.

fix broken links
2 participants