Skip to content

Add footnotes clarifying visitor counting and data attribution#223

Merged
CodyCBakerPhD merged 2 commits into
mainfrom
claude/unique-visitors-footnote-7813lo
Jul 13, 2026
Merged

Add footnotes clarifying visitor counting and data attribution#223
CodyCBakerPhD merged 2 commits into
mainfrom
claude/unique-visitors-footnote-7813lo

Conversation

@CodyCBakerPhD

Copy link
Copy Markdown
Collaborator

Summary

Added clarifying footnotes to the usage statistics header to provide users with important context about how unique visitors are counted and how Dandiset source determination works.

Key Changes

  • Added a dagger symbol (<sup>&dagger;</sup>) next to "unique visitors" in the totals header to reference a new footnote
  • Added a new footnote explaining that unique visitors are counted by IP address and may be affected by VPN usage patterns
  • Reorganized the footnote text to include both the existing asterisk note about Dandiset source determination and the new dagger note about visitor counting methodology

Implementation Details

  • The footnote HTML is now constructed by concatenating two explanatory statements
  • Both footnotes are displayed in the same small-text footer element (0.5em font size with 7px top margin)
  • The changes maintain the existing styling and layout of the statistics display

https://claude.ai/code/session_01GtAPTPk6UvFi83whnfiW8T

…skewed by VPN usage

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GtAPTPk6UvFi83whnfiW8T
@CodyCBakerPhD CodyCBakerPhD self-assigned this Jul 13, 2026
@CodyCBakerPhD CodyCBakerPhD marked this pull request as ready for review July 13, 2026 19:41
@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://dandi.github.io/usage-page/pr-preview/pr-223/

Built to branch gh-pages at 2026-07-13 19:43 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@codecov

codecov Bot commented Jul 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 11.88%. Comparing base (a0df47d) to head (db14a0a).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/plots.ts 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #223   +/-   ##
=======================================
  Coverage   11.88%   11.88%           
=======================================
  Files           4        4           
  Lines        1220     1220           
  Branches      308      308           
=======================================
  Hits          145      145           
  Misses       1075     1075           
Flag Coverage Δ
unit 11.88% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/plots.ts 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GtAPTPk6UvFi83whnfiW8T
@CodyCBakerPhD CodyCBakerPhD merged commit 7453646 into main Jul 13, 2026
16 checks passed
@CodyCBakerPhD CodyCBakerPhD deleted the claude/unique-visitors-footnote-7813lo branch July 13, 2026 19:47
@CodyCBakerPhD CodyCBakerPhD moved this to Uncategorized in My Work History Jul 14, 2026
@github-project-automation github-project-automation Bot moved this from Uncategorized to Done in My Work History Jul 14, 2026
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