Skip to content

Update donation CTA model with new copy#702

Merged
jagerman merged 3 commits into
session-foundation:devfrom
jagerman:ongoing-dev-cta
Jul 9, 2026
Merged

Update donation CTA model with new copy#702
jagerman merged 3 commits into
session-foundation:devfrom
jagerman:ongoing-dev-cta

Conversation

@jagerman

@jagerman jagerman commented Jul 6, 2026

Copy link
Copy Markdown
Member
  • Rename the modal variant .finalAppeal -> .ongoingAppeal and point it at
    the new ongoingAppeal / ongoingAppealDescription string keys (new keys so
    stale finalAppeal* translations can't resurface).
  • Swap the confirm button to readMoreCapital ("Read More") to match Android.
  • Replace the hero image (SessionHeroWeb.webp).
  • Bump donationTriggersResetDate to re-trigger the modal for existing users.
  • Bump version to 2.15.3 (build 722).

The ongoingAppeal* keys are supplied by the Crowdin sync (#701); this PR includes those changes (needed for successful compilation).

Also updates ARCHITECTURE.md with translation generation details, which weren't mentioned at all.

@mpretty-cyro mpretty-cyro changed the base branch from master to dev July 6, 2026 20:12
jagerman and others added 2 commits July 9, 2026 14:21
Add a new "Localization and Translations" section (§19) describing the
Crowdin-managed string pipeline: source of truth, the sync automation in
session-shared-scripts, approved-only export, and the merge ordering
required when shipping a text change. Renumber subsequent sections.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- Rename the modal variant .finalAppeal -> .ongoingAppeal and point it at
  the new ongoingAppeal / ongoingAppealDescription string keys (new keys so
  stale finalAppeal* translations can't resurface).
- Swap the confirm button to readMoreCapital ("Read More") to match Android.
- Replace the hero image (SessionHeroWeb.webp).
- Bump donationTriggersResetDate to re-trigger the modal for existing users.
- Bump version to 2.15.3 (build 722).

Note: the ongoingAppeal* string keys must land via the Crowdin sync PR
before this compiles (LintLocalizableStrings fails on missing keys).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@jagerman jagerman merged commit eec26c9 into session-foundation:dev Jul 9, 2026
1 check was pending
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