Skip to content

Update donation CTA with new copy and image#2139

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

Update donation CTA with new copy and image#2139
jagerman merged 2 commits into
session-foundation:devfrom
jagerman:ongoing-dev-cta

Conversation

@jagerman

@jagerman jagerman commented Jul 6, 2026

Copy link
Copy Markdown
Member

Refresh the donation call-to-action shown on the home screen:

  • point DonationDialog at the new ongoingAppeal / ongoingAppealDescription
    string keys (the "New hope for Session" copy) instead of finalAppeal*
  • replace the hero image (cta_hero_donation.webp)
  • bump the donation preference keys v3 -> v4 so the refreshed CTA is shown
    again to users who had already dismissed the previous one

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

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

@jagerman jagerman marked this pull request as draft July 6, 2026 18:57
jagerman and others added 2 commits July 6, 2026 17:24
Add a Localization and Translations section (after UI Architecture) explaining
that strings.xml files are generated from Crowdin and must not be hand-edited,
that the sync automation lives in session-shared-scripts, the approved-only
export gate, and the ordering for shipping a text change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Refresh the donation call-to-action shown on the home screen:

- point DonationDialog at the new ongoingAppeal / ongoingAppealDescription
  string keys (the "New hope for Session" copy) instead of finalAppeal*
- replace the hero image (cta_hero_donation.webp)
- bump the donation preference keys v3 -> v4 so the refreshed CTA is shown
  again to users who had already dismissed the previous one

The ongoingAppeal* keys are supplied by the Crowdin sync; this will not
compile until that translations PR lands them in strings.xml.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jagerman jagerman marked this pull request as ready for review July 6, 2026 20:27
@jagerman jagerman changed the base branch from master to dev July 6, 2026 20:28
@jagerman jagerman merged commit 1e8d01c into session-foundation:dev Jul 9, 2026
10 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