Skip to content

top coin holders to metrics - #440

Merged
aaravjhamb merged 1 commit into
mainfrom
fix-journal-date-disambiguation
Sep 10, 2026
Merged

top coin holders to metrics#440
aaravjhamb merged 1 commit into
mainfrom
fix-journal-date-disambiguation

Conversation

@aaravjhamb

@aaravjhamb aaravjhamb commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Adds a top 100 coin holders table to the metrics page, ranked by current balance with earned/adjustments/spent alongside. Reuses the per-user totals that the existing "in accounts now" number already needed, so it doesn't cost an extra pass over projects. The journal date fix that was on this branch moved to #441.

@aaravjhamb aaravjhamb changed the title Disambiguate journal dates and add top coin holders top coin holders to metrics Sep 10, 2026
Comment thread app/jobs/sync_journal_job.rb Outdated

date = build_safe_date(m[3]&.to_i || today.year, month, m[1].to_i, today)
next unless date
return date unless ENTRY_COUNTER_PATTERN.match?(source[0...m.begin(0)])
@aaravjhamb
aaravjhamb force-pushed the fix-journal-date-disambiguation branch from 5e75d9a to c3b9e3d Compare September 10, 2026 03:47
@aaravjhamb
aaravjhamb merged commit 303895b into main Sep 10, 2026
6 of 11 checks passed
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