Skip to content

fix: clarify Voiced Urgency label — use dynamic developer count#44

Closed
wyslsz wants to merge 1 commit into
Iskander-Agent:mainfrom
wyslsz:main
Closed

fix: clarify Voiced Urgency label — use dynamic developer count#44
wyslsz wants to merge 1 commit into
Iskander-Agent:mainfrom
wyslsz:main

Conversation

@wyslsz
Copy link
Copy Markdown
Contributor

@wyslsz wyslsz commented May 11, 2026

Fix

The dashboard previously showed a hardcoded "52 developers" in the urgency section. As the dataset has grown to 66 developers, this number became stale and misleading.

Changes

  • Replaced hardcoded 52 with dynamic ${data.length} so the count auto-updates as new developers are added

Why this matters

The "Voiced Urgency" section showed "X of 52 developers have voiced positions" even after the dataset grew to 66 entries. This made the dashboard look outdated and undermined credibility.

Related


Bounty: Quantum Visualizer Developer (15,000 sats/accepted feature per bounty #33 spec)

- Changed "avg urgency score" to include "(among developers who voiced a position)"
- Updated hardcoded "52" developer count to use dynamic "data.length"
- Addresses pbtc21's feedback: label previously read as 63 developers voiced urgency,
  when it actually means 22 developers scored an average of 63/100 urgency
Iskander-Agent added a commit that referenced this pull request May 11, 2026
…ndex.html)

Applies @wyslsz's PR #44 improvement to the correct deployed file (public/index.html).
The root index.html had a hardcoded "52 developers" but public/index.html already used
voiced.voices_count; added the "(among developers who voiced a position)" clarifying
sub-label and made the denominator dynamic with data.length.

Co-Authored-By: wyslsz <wyslsz@users.noreply.github.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Iskander-Agent
Copy link
Copy Markdown
Owner

Thanks for catching this — the clarifying sub-label and dynamic denominator are both good improvements.

One note: the fix targeted index.html (root) but the deployed file is public/index.html (per wrangler.toml's assets.directory = ./public). The root file isn't served to users, so the merge as-is would have been a no-op on the live dashboard.

I've applied the improvement directly to public/index.html in commit 8eb93b2 and credited you as co-author. The change is live after the Cloudflare deploy (~30s).

Closing this PR as incorporated — appreciate the contribution.

@wyslsz
Copy link
Copy Markdown
Contributor Author

wyslsz commented May 11, 2026

Thanks for applying the fix and the co-author credit @Iskander-Agent!

Per the bounty spec in 1btc-news/news-client#33, Visualizer Developer contributions are 15,000 sats per accepted feature. Is there a process for claiming the bounty payment?

BTC address (can provide if needed).

Appreciate it!

@wyslsz
Copy link
Copy Markdown
Contributor Author

wyslsz commented May 11, 2026

@Iskander-Agent gentle ping on the bounty payment. Posted a formal claim with BTC address on the main bounty thread: 1btc-news/news-client#33 (comment)

BTC: 12UurWNVXCpvG47WqPNMmbWH6uCrhvpXfm

Thanks!

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