Skip to content

fix(rewards): prefill Reddit and LinkedIn share links#952

Merged
lefarcen merged 3 commits intomainfrom
fix/share-prefill-links
Apr 8, 2026
Merged

fix(rewards): prefill Reddit and LinkedIn share links#952
lefarcen merged 3 commits intomainfrom
fix/share-prefill-links

Conversation

@lefarcen
Copy link
Copy Markdown
Collaborator

@lefarcen lefarcen commented Apr 8, 2026

What

Update the rewards share links so Reddit opens with a prefilled link-post title and LinkedIn opens with prefilled share copy plus the target URL.

Why

The existing Reddit and LinkedIn links only passed the bare URL, which made the share flow feel incomplete compared with the other social tasks and with competing invitation flows.

How

  • add title and type=LINK to the Reddit submit URL
  • switch LinkedIn from share-offsite to the feed share URL with text and shareUrl
  • add a shared test that asserts those prefill parameters are present

Affected areas

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

Checklist

  • pnpm typecheck passes (not run)
  • pnpm lint passes (verified via pnpm exec biome check on touched files)
  • pnpm test passes (verified via pnpm exec vitest run tests/shared/rewards-proof.test.ts)
  • 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

These URL shapes are intentionally based on currently working web share entrypoints rather than formal API integrations, so manual spot-checking in browser is worthwhile.

@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@lefarcen lefarcen merged commit f8659ac 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.

3 participants