v8 stack 5/5: docs and example app - #27
Closed
dmurphy5 wants to merge 1 commit into
Closed
Conversation
README covers the New Architecture requirement, the durable journal and the read/ack cycle, typed errorKind and cancelReason, acceptStatus, getAllUploads, the AppDelegate hook for background session completion, and the fact that pod install now runs codegen. Adds a note that consumers must resolve this package through Metro rather than a nested node_modules. CHANGELOG records the breaking changes. The example app is updated to the new API so it stays a working reference. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This was referenced Jul 27, 2026
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 stack
Five PRs, each based on the one before it. This is 5/5.
ai.openspacerenamemaster#22 stays open and untouched as the single-diff view of the same change.
Its tree and this stack's tip are byte-identical (tree
f8404b0), so mergingeither one produces exactly the same
master. Review whichever is easier tofollow; merge one and close the other.
Documentation and the example app, closing out the stack.
README covers the New Architecture requirement, the durable journal and its
read/ack cycle, typed
errorKindandcancelReason,acceptStatus,getAllUploads, the AppDelegate hook for background session completion, and thefact that
pod installnow runs codegen. Adds a note that consumers must resolvethis package through Metro rather than a nested
node_modules— a linked cloneotherwise ships a second copy of React and Metro resolves the wrong one.
CHANGELOG records the breaking changes. The example app moves to the new API so
it stays a working reference.
This is the tip of the stack
The tree at this commit is byte-identical to
dylan/phase0-reliable-uploads(#22) — same tree hash,
f8404b0. Merging the stack and merging #22 produceexactly the same
master.Verified at the tip
yarn lint:ci,yarn typecheck,yarn test(7/7), and:react-native-background-upload:testDebugUnitTest(15/15) all pass.