docs(mobile): refresh iOS/Android SDK guides to the shipped 0.2.3 API#188
Merged
Conversation
The mobile guides had drifted from the actual SDK surface (verified against the generated bindings and OrangeRock's real integration): - `reportAttributionForLink` does not exist — the method is `attributeLink`. Fixed in the iOS + Android API tables and the Android install-referrer / deferred snippets (the lifecycle page already used the correct name). - `parseClipboardLink(text:)` now takes `allowedHosts:` (shipped in 0.2.2). - iOS guide: state the iOS 15 minimum and recommend SPM via the repo URL (how OrangeRock consumes it); keep the tarball as a fallback. - iOS methods table: add the recommended `checkDeferredDeepLinkFromPasteboard`. - Add a "Handle an incoming link" section to both guides for the already-installed case (iOS `.onOpenURL`, Android App Link intent → `attributeLink` + `getLink`), mirroring OrangeRock's deep-link routing. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
1 participant
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 mobile SDK guides had drifted from the actual API. Verified each change against the generated UniFFI bindings (
rift_ffi.swift) and OrangeRock's real integration.Fixes (stale → correct)
reportAttributionForLink→attributeLink. That name doesn't exist in the bindings (0 occurrences); the real method isattributeLink(linkId:). Fixed in the iOS + Android API tables and the Android install-referrer / deferred snippets. (The lifecycle page already used the correct name — the pages disagreed.)parseClipboardLink(text:)→parseClipboardLink(text:, allowedHosts:)— the host-validation arg shipped in 0.2.2.Package.swift.v15from fix(sdk): repair iOS pasteboard wrapper + compile Swift for iOS in CI (0.2.3) #187).Alignment with real-world usage
https://github.com/saltyskip/rift(how OrangeRock consumes it), with the tarball as a fallback..onOpenURL, Android App Link intent →attributeLink+getLink, mirroring OrangeRock's deep-link routing. (Deferred deep linking only covered the not-installed case.)checkDeferredDeepLinkFromPasteboard(clearOnMatch:)alongside the lower-levelcheckDeferredDeepLink(clipboardText:).Notes
marketing/src/app/docs/{ios-sdk,android-sdk,deferred}/page.tsx). Manually reviewed JSX for escaped entities; the marketing lint/build runs in its own pipeline (deps weren't installed in this worktree).detectPatternscallout,setUserId/clearUserId/trackConversionsignatures, storage notes.🤖 Generated with Claude Code