Skip to content

API docs link gives 404 #510

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

Closed
lorepieri8 opened this issue Sep 2, 2021 · 2 comments · Fixed by #533
Closed

API docs link gives 404 #510

lorepieri8 opened this issue Sep 2, 2021 · 2 comments · Fixed by #533
Assignees
Labels

Comments

@lorepieri8
Copy link

Describe the bug

API docs link gives 404

To Reproduce

https://ros-controls.github.io/control.ros.org/ros2_control/doc/index.html
Click on "here"

Expected behavior
A clear and concise description of what you expected to happen.

@lorepieri8 lorepieri8 added the bug label Sep 2, 2021
@bmagyar
Copy link
Member

bmagyar commented Sep 2, 2021 via email

@jaron-l
Copy link
Member

jaron-l commented Sep 13, 2021

Looks like I messed up the link on the API documentation page to the doxygen sub-path and it needs the extra controls.ros.org sub-folder path. So it should point to something like https://ros-controls.github.io/control.ros.org/api/index.html instead of https://ros-controls.github.io/api/index.html which doesn't exist. I'm going to investigate this a little more to find something that will hopefully work with both the local build and the dynamic one that GitHub pages makes.

@jaron-l jaron-l self-assigned this Sep 13, 2021
jaron-l added a commit to jaron-l/ros2_control that referenced this issue Sep 16, 2021
* Change url for Doxygen pages to be from root to a relative link

Fixes ros-controls#510
destogl pushed a commit that referenced this issue Sep 16, 2021
…533)

* Change url for Doxygen pages to be from root to a relative link

Fixes #510
pac48 pushed a commit to pac48/ros2_control that referenced this issue Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants