Skip to content

New testing tool and fix broken links#10

Merged
iluetkeb merged 3 commits into
masterfrom
feature/web_updates
Feb 28, 2019
Merged

New testing tool and fix broken links#10
iluetkeb merged 3 commits into
masterfrom
feature/web_updates

Conversation

@BorjaOuterelo

@BorjaOuterelo BorjaOuterelo commented Feb 28, 2019

Copy link
Copy Markdown
Contributor

Added a useful HTML testing tool.
First, run already pointed out broken links. Fix for those is also provided in this PR

@iluetkeb

iluetkeb commented Feb 28, 2019 via email

Copy link
Copy Markdown
Contributor

@BorjaOuterelo

Copy link
Copy Markdown
Contributor Author

I changed them because going through the links present in https://microros.github.io/docs/home/ I got 404 for all of them.
Going through the links present there work for you?
That may be an issue on my side. I am missing something?.

@iluetkeb

iluetkeb commented Feb 28, 2019

Copy link
Copy Markdown
Contributor

These links really don't work, even though they worked when running Jekyll locally on my laptop. Apparently, something with the Jekyll-redirect plugin is wrong. We have to investigate that, because we put theses links into deliverables.
I've tested it now and https://microros.github.io/embedded_tf works, and redirects to .../docs/embedded_tf/. So, unclear why you got 404s!

@iluetkeb iluetkeb merged commit 019b702 into master Feb 28, 2019
@BorjaOuterelo BorjaOuterelo deleted the feature/web_updates branch March 6, 2019 06:54
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