Skip to content

Show view counts on the shareable resource pages - #583

Merged
Alejandro Lopez (soyalejolopez) merged 1 commit into
masterfrom
show-views-on-resource-pages
Nov 2, 2026
Merged

Alejandro Lopez (soyalejolopez) merged 1 commit into
masterfrom
show-views-on-resource-pages

Conversation

@soyalejolopez

Copy link
Copy Markdown
Member

Each r// page now has a Views row in its details panel, between Format and Version, matching the catalog's detail view. The page reads resource-stats.json in the browser, so counts update without rebuilding the pages. The row stays hidden if the file cannot be read, and download pages do not have one. A resource with no recorded views reads 0.

  • Hover text is the same as the catalog's, and a test fails if the two drift apart.
  • Tests cover the row's position, the script's source and slug lookup, the hidden-row CSS, and the absence of the row on download pages.
  • Regenerated the 58 resource pages.

Each r/<slug>/ page now has a Views row in its details panel, between Format and Version, matching
the catalog's detail view. The page reads resource-stats.json in the browser, so counts update
without rebuilding the pages. The row stays hidden if the file cannot be read, and download pages
do not have one. A resource with no recorded views reads 0.

- Hover text is the same as the catalog's, and a test fails if the two drift apart.
- Tests cover the row's position, the script's source and slug lookup, the hidden-row CSS, and
  the absence of the row on download pages.
- Regenerated the 58 resource pages.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

🛰️ PR Sweeper report

Risk: 🟢 LOW · Security gate: ✅ passing · Files: 62

🔒 Automated guardrails (authoritative)

Category Location Finding
ℹ️ scope — Change(s) outside known content roots: assets/resource-page.css, assets/resource-page.js, r/add-engagegroupadmins/index.html, r/agent-365-enablement-checklist/index.html, r/agent-365-lifecycle-atlas/index.html, r/agent-brainstorming-template/index.html, r/agents-cost-calculator/index.html, r/ai-council/index.html, r/analyze-sharepointrisk/index.html, r/autoreply-agent/index.html ….
ℹ️ scope — Large PR: 62 files changed. Smaller PRs are easier to review.

🤖 Dual-model AI review (advisory)

AI review unavailable for this run (models not reachable or no diff). Guardrails above are unaffected.

  • GPT-class reviewer (openai/gpt-5): skipped — request failed: Unexpected token 'O', "OK
    " is not valid JSON
  • Deep-reasoning reviewer (openai/o3): skipped — request failed: Unexpected token 'O', "OK
    " is not valid JSON

The automated guardrails are authoritative and gate the security status. The AI review is advisory and never auto-merges. Thanks for contributing to FastTrack! 🛩️

@github-actions github-actions Bot added sweeper:risk-low PR Sweeper: low risk sweeper:scope-review PR Sweeper: scope/sensitive-path review labels Nov 2, 2026
@soyalejolopez
Alejandro Lopez (soyalejolopez) merged commit 2630af5 into master Nov 2, 2026
8 checks passed
@soyalejolopez
Alejandro Lopez (soyalejolopez) deleted the show-views-on-resource-pages branch November 2, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweeper:risk-low PR Sweeper: low risk sweeper:scope-review PR Sweeper: scope/sensitive-path review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant