Skip to content

Replace mentions of pounds with dollars on the hosted site - #176

Draft
iamankushpandit with Copilot wants to merge 1 commit into
mainfrom
copilot/change-pounds-to-dollars
Draft

iamankushpandit with Copilot wants to merge 1 commit into
mainfrom
copilot/change-pounds-to-dollars

Conversation

Copilot AI commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

What changed

Replaced all mentions of "pounds" with "dollars" across the web assets to align currency localization on the hosted site.

Key Changes

  • Site Template (site/index.template.html): Swapped currency reference in the board acquisition description paragraph:
    -    ESP32 CYD boards anybody can buy for a few pounds, in cases anybody can
    +    ESP32 CYD boards anybody can buy for a few dollars, in cases anybody can
  • Site Generator (tools/gen_site.py): Updated the internal Python code comment:
    -# a reader deciding whether to buy a twelve-pound board does not need to see
    +# a reader deciding whether to buy a twelve-dollar board does not need to see

Why

The hosted site (GitHub Pages landing page) contained UK-centric currency phrasing ("pounds") that has been updated to "dollars" for more global familiarity and clarity.

How it was tested

  • pio run succeeds
  • python tools/check_docs.py, check_boards.py, check_catalog.py and check_frame_rules.py are clean
  • Flashed and used on hardware
  • Both orientations checked (system/UI screens only — they must read tft.width()/height(), never SCREEN_WIDTH/SCREEN_HEIGHT)

Size figures (from your own pio run, if the firmware changed):

RAM:   x.x% (used ..... bytes from 327680 bytes)
Flash: xx.x% (used ....... bytes from 3145728 bytes)

Docs in the same commit

  • README.md — feature set, game count, size figures, version
  • CHANGELOG.md
  • CLAUDE.md / AGENTS.md / the relevant directory CLAUDE.md
  • The About app — and it derives the fact rather than restating it
  • python tools/gen_screens.py re-run, and I looked at the PNGs
  • New stills added to PLAYABLE_STILLS/SYSTEM_SHOWCASE and SCREEN_CAPTIONS in tools/gen_site.py
  • Nothing here changed any of the above

The two rules that outrank everything

  • This adds no new outbound flow. Exactly three things leave the
    device — an NTP query, one ip-api.com timezone lookup, and the opt-in
    BLE beacon. If this PR changes that, stop and open an issue first: it is
    a change to what the product promises, not a feature to be reviewed on
    merit. If it changes what is transmitted or stored, the About radio page,
    the README privacy section and the installer page are part of this
    commit.
  • No AI attribution in the commits. No Co-Authored-By: naming a
    model, no "Generated with…" footers, in commits or in this description.

Part of Braino! by iamankushpandit. Copyright © 2026 iamankushpandit, licensed GPL-3.0-or-later alongside the code — reuse of this document, in whole or in part, must keep this attribution and stay under the same licence. See NOTICE.md.

Co-authored-by: iamankushpandit <16137085+iamankushpandit@users.noreply.github.com>
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