-
Notifications
You must be signed in to change notification settings - Fork 0
docs: make the repo landing page convert - #128
Merged
Merged
Conversation
The same raw Expo token can be owned by several users — (user_id, token) uniqueness allows it deliberately, since the old UNIQUE(token) plus ON CONFLICT DO UPDATE SET user_id was a push-hijack hole (migration 000047). ListAllPushTokens returned every row, so broadcast_push built one message per row and a device shared by two accounts got the release note twice. Observed in prod: one iPhone registered under both the App Store review account and its real owner received the 1.4.2 broadcast twice. DISTINCT ON (token), most-recently-used row first, so the device is attributed to the account actually signed in on it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VWWp1xZ8BsBNdkNQq259Xa
The repo read as abandoned to a first-time visitor: no screenshots, no store links, a placeholder description, no topics, and a status doc six weeks stale that listed shipped features as deferred. - README: hero with the five App Store screenshots, store/website links, a comparison table against Splitwise and the web-only OSS alternatives, and the self-host command moved above the repo-layout tables. - docs/images/screenshots/: the ten App Store screenshots, committed so the README does not hotlink Apple's CDN. - implementation-status.md: activity feed, attachments, the Splitwise importer and Google/Apple sign-in have all shipped; move them out of the deferred list and refresh the date. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VWWp1xZ8BsBNdkNQq259Xa
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The repo read as abandoned to a first-time visitor: no screenshots, no store links, a placeholder description (
Now we're kvitt), no topics, and a status doc six weeks stale that listed shipped features as deferred. Every launch channel sends people to the repo, so the repo is what converts.Changes
docs/images/screenshots/— the ten App Store screenshots, committed rather than hotlinked from Apple's CDN.docs/implementation-status.md— activity feed UI, image attachments, the Splitwise importer and Google/Apple sign-in have all shipped; moved out of the deferred list. Date refreshed.Competitor claims I could not verify (receipt scanning and multi-currency in Spliit / SplitPro) say "varies" rather than a false ❌.
Done outside the diff
Repo settings, applied directly: real description, homepage set to the marketing site, 18 topics (
splitwise-alternative,self-hosted,bill-splitting, ...), and Discussions enabled.Still needs a human
marketing/public/og.jpgas the repo social preview in Settings. It is what renders when the link is shared on Hacker News, Reddit and elsewhere.good first issue— the README now links that filter and it is currently empty.🤖 Generated with Claude Code
https://claude.ai/code/session_01VWWp1xZ8BsBNdkNQq259Xa