Copilot review on the cache commit raised three points: 1. inline.js had a hard-coded `https://api.github.com/repos/umputun/remark42` while the templates use `site.githubUrl`. Rename inline.js → inline.njk so nunjucks evaluates it, add `githubApiUrl` to site.json, and template the fetch URL from it. One place to update if the repo ever moves. 2. header.njk aria-label said "Remark42's GitHub Repository" but the link target is `/releases`. Change to "{{ site.name }} releases on GitHub" so screen readers describe the actual destination. 3. console.warn on fetch failure (kept after umputun's prior review noted the trade-off): addressed in the PR description, no code change.
11 lines
528 B
JSON
11 lines
528 B
JSON
{
|
|
"name": "Remark42",
|
|
"subtitle": "Privacy focused lightweight commenting system",
|
|
"description": "Remark42 is a self-hosted, lightweight, and simple (yet functional) comment engine, which doesn't spy on users. It can be embedded into blogs, articles or any other place where readers add comments.",
|
|
"url": "https://remark42.com",
|
|
"githubUrl": "https://github.com/umputun/remark42",
|
|
"githubApiUrl": "https://api.github.com/repos/umputun/remark42",
|
|
"githubBranch": "master",
|
|
"remark42Host": "https://demo.remark42.com"
|
|
}
|