Skip to content

fix(rewards): refine mobile invite copy and WhatsApp proof flow#945

Merged
lefarcen merged 2 commits intomainfrom
fix/rewards-copy-reddit-share
Apr 8, 2026
Merged

fix(rewards): refine mobile invite copy and WhatsApp proof flow#945
lefarcen merged 2 commits intomainfrom
fix/rewards-copy-reddit-share

Conversation

@lefarcen
Copy link
Copy Markdown
Collaborator

@lefarcen lefarcen commented Apr 8, 2026

What

Refine rewards copy around mobile invites and earned credits, fix the LinkedIn label, and stop requiring a pasted share URL for WhatsApp reward claims.

Why

The current copy still references specific mobile apps and internal credit-consumption details, the Chinese LinkedIn label has a typo, and WhatsApp sharing often does not produce a public URL that users can paste back into the rewards flow.

How

  • update the zh-CN and en rewards copy for mobile invite and earned-credit tooltips
  • rename the LinkedIn task label to the correct wording
  • remove WhatsApp from the URL-proof task set and adjust related tests

Affected areas

  • Desktop app (Electron shell)
  • Controller (backend / API)
  • Web dashboard (React UI)
  • OpenClaw runtime
  • Shared schemas / packages
  • Skills
  • Build / CI / Tooling

Checklist

  • pnpm typecheck passes (blocked in this worktree: local toolchain missing, tsc not found)
  • pnpm lint passes (blocked in this worktree: biome not found)
  • pnpm test passes (blocked in this worktree: vitest not found)
  • pnpm generate-types run (if API routes/schemas changed)
  • No credentials or tokens in code or logs
  • No any types introduced (use unknown with narrowing)

Notes for reviewers

This PR intentionally does not change LinkedIn/Facebook share prefill behavior; it only tightens copy and removes the WhatsApp pasted-link requirement.

@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 8, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 6 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
apps/web/src/i18n/locales/en.ts 0.00% 6 Missing ⚠️

📢 Thoughts on this report? Let us know!

@lefarcen lefarcen merged commit b1a3065 into main Apr 8, 2026
11 checks passed
@lefarcen lefarcen mentioned this pull request Apr 8, 2026
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