-
-
Notifications
You must be signed in to change notification settings - Fork 245
Conversation
@@ -2,8 +2,8 @@ | |||
<div class="container"> | |||
<footer> | |||
<ul class="list-inline"> | |||
{% if page.edit_link %} | |||
<li><a href="{{ page.edit_link }}">Edit this page</a></li> | |||
{% if edit_link %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The root of the problem - this data is at the top level, not nested in page
@@ -1,7 +1,6 @@ | |||
--- | |||
title: 404 | |||
layout: doc | |||
edit_link: https://github.com/eslint/eslint.github.io/edit/master/_pages/404.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are generated by line 8 of _includes/footer.liquid
,
@@ -1,5 +1,5 @@ | |||
--- | |||
title: ESLint Contributor License Agreement | |||
layout: content-only | |||
edit_link: https://github.com/eslint/eslint.github.io/edit/master/_pages/cla.md | |||
edit_link: https://github.com/eslint/website/edit/master/_pages/cla.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This redirects correctly, but we might as well update it.
2577157
to
5d89cba
Compare
5d89cba
to
2577157
Compare
2577157
to
f54384b
Compare
Deploy preview for eslint ready! Built with commit 2577157 |
Deploy preview for eslint ready! Built with commit f54384b |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice 👏
As reported here, there are a number of broken "Edit this page" links. This PR fixes them!