fix: clean shared links + add copy-link button#59
Conversation
- shared links no longer carry a redundant text body: the route share
dropped `text: "Pumperly route"` and the station share dropped
`text: "<brand> — <price>"`. Native share targets prepend `text` to the
URL, which is what appended "Pumperly route " to shared route links.
Now shares { title, url } only.
- station popup: add a dedicated Copy-link button (clipboard icon → check)
next to Share that copies the station deep-link directly without opening
the share sheet. Exposes copyToClipboard() from lib/share.ts.
- i18n: popup.copyLink across all 16 locales.
|
Warning Review limit reached
More reviews will be available in 33 minutes and 14 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Two small follow-ups to the v1.6.0 sharing feature, from live feedback:
text: "Pumperly route"and the station sharetext: "<brand> — <price>". Native share targets prepend thetextfield to the URL, so shared route links arrived asPumperly route https://…— the "extra words at the end" the user reported. Now both share{ title, url }only (title populates the share-sheet heading without polluting the link).copyToClipboard()is now exported fromlib/share.ts.popup.copyLinkacross all 16 locales.Test plan