Skip to content

Commit 28834f6

Browse files
authored
Fix: Remove "Signal issue" links from docs
Fix #962 Close #966
1 parent 04b962f commit 28834f6

File tree

2 files changed

+0
-15
lines changed

2 files changed

+0
-15
lines changed

src/webhint-theme/helper/utils.js

-8
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
const pagination = require('./pagination');
22

3-
const url = require('url');
4-
53
const isIndexPage = (page) => {
64
return page.source.endsWith('index.md');
75
};
@@ -77,12 +75,6 @@ module.exports = {
7775
return page.section === section;
7876
});
7977
},
80-
getSignalIssueQuery: (root, title, directory) => {
81-
const issueTitle = `[docs] Issue with '${title}'`;
82-
const issueContent = url.resolve(root, directory);
83-
84-
return `title=${encodeURIComponent(issueTitle)}&body=${encodeURIComponent(issueContent)}`;
85-
},
8678
/**
8779
* Return the `href` value to link to the target page from the current one.
8880
* for example when the current page is `hint-amp-validator`:

src/webhint-theme/layout/partials/doc.ejs

-7
Original file line numberDiff line numberDiff line change
@@ -36,13 +36,6 @@ toCItems = utils.getToCItems(pages); %>
3636
>Edit</a
3737
>
3838
</li>
39-
<li>
40-
<a
41-
target="_blank"
42-
href="<%= `${theme.webhintRepoRoot}/issues/new?${utils.getSignalIssueQuery(theme.webhintRoot, page.title, page.permalink)}` %>"
43-
>Signal Issue</a
44-
>
45-
</li>
4639
</ul>
4740
</div>
4841
</header>

0 commit comments

Comments
 (0)