Skip to content

docs: README updates (trust section, badges, release notes) - #256

Merged
kcw-grunt merged 1 commit into
mainfrom
claude/docs
Jul 29, 2026
Merged

docs: README updates (trust section, badges, release notes)#256
kcw-grunt merged 1 commit into
mainfrom
claude/docs

Conversation

@grunt-claude-bot

Copy link
Copy Markdown
Collaborator

Summary

Cherry-pick of 7f6aaec (#254) directly onto a branch cut from main, so this PR's diff/commit list contains only that one commit instead of develop's full divergent history from main (see discussion on #255).

  • Added a "Why Brainwallet" trust section (SPV/serverless, BIP32/38/70, Android Keystore) with claims verified against this codebase
  • Play Store badge in its own subsection (resized to 200px), plus release-version and CircleCI (main/develop) badges
  • Release Notes updated through v4.10.6, with standardized **vX.Y.Z** [PR [#N]] headers

Test plan

  • N/A — docs-only change

…at (#254)

Co-authored-by: grunt-claude-bot <308083480+grunt-claude-bot@users.noreply.github.com>

@kcw-grunt kcw-grunt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kcw-grunt
kcw-grunt merged commit 3f2db9d into main Jul 29, 2026
3 checks passed
kcw-grunt pushed a commit that referenced this pull request Jul 31, 2026
commit f701d2d
Author: grunt-claude-bot <admin@brainwallet.co>
Date:   Fri Jul 31 08:55:58 2026 +0100

    Activate InAppReviewService and wire up review prompts (#260)

    * di: register InAppReviewService in AppModule

    Wires the previously-dormant InAppReviewService into Koin using the
    existing BrainwalletApp.breadContext activity tracker. No call sites
    added yet — showInAppReviewDialogIfNeeded() is not invoked anywhere,
    so behavior is unchanged until it's called from the app.

    * feat: activate InAppReviewService and wire up review prompts

    Registers InAppReviewService in Koin DI and calls
    showInAppReviewDialogIfNeeded() from four placements: after a
    successful send, the game hub exit flow, the balance-visibility
    toggle, and two tutorial pages. Fixes several placement bugs found
    along the way (broken suspend-context usage, unresolved DI
    references, malformed syntax, and an unreachable branch in
    GameHubViewModel), and adds unit test coverage for the service's
    gating and Play Core success/failure branches.

commit 1a71dad
Author: grunt-claude-bot <admin@brainwallet.co>
Date:   Thu Jul 30 16:42:24 2026 +0100

    docs: add v4.11.0 release notes to README

commit faac661
Merge: 07224b9 3f2db9d
Author: Kerry Washington <mrkerrywashington@icloud.com>
Date:   Thu Jul 30 16:14:18 2026 +0100

    Merge branch 'main' into release/v4.11.0

commit 07224b9
Author: grunt-claude-bot <admin@brainwallet.co>
Date:   Thu Jul 30 16:10:21 2026 +0100

    test: add unit test coverage for pure utility helpers and gameinterface logic

    Covers previously-untested Base58, TypesConverter, BytesUtil, BRCompressor,
    and BRDateUtil classes, plus the GameKoinModule DI wiring and GdxGameView's
    game-exit JSON parsing/social-share dispatch logic (GameExitData and
    handleGameExit widened from private to internal to make them testable).

commit 3f2db9d
Author: grunt-claude-bot <admin@brainwallet.co>
Date:   Wed Jul 29 14:17:53 2026 +0100

    docs: standardize release headers to bold version + PR + divider format (#254) (#256)

    Co-authored-by: grunt-claude-bot <308083480+grunt-claude-bot@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