Skip to content

feat: re-write to remove deprecated apollo syntax#544

Closed
kieran-osgood-shopify wants to merge 4 commits intokieran-osgood/03-25-featbumpapollotov2from
kieran-osgood/03-25-featre-writetoremovedeprecatedapollosyntax
Closed

feat: re-write to remove deprecated apollo syntax#544
kieran-osgood-shopify wants to merge 4 commits intokieran-osgood/03-25-featbumpapollotov2from
kieran-osgood/03-25-featre-writetoremovedeprecatedapollosyntax

Conversation

@kieran-osgood-shopify
Copy link
Copy Markdown
Contributor

@kieran-osgood-shopify kieran-osgood-shopify commented Mar 25, 2026

What changes are you making?

Previous PR bumped to apollo v2 but didn't resolve deprecation warnings
This PR replaces deprecated v1 apollo apis with their v2 equivalent
Apollo v2 gives native async/await support and concurrency handling, meaning we can remove the wrappers like withCheckedThrowingContinuation


Before you merge

Important

Releasing a new version of the kit?

Releasing a new major version?

  • I have bumped the version number in the README.

Tip

See the Contributing documentation for instructions on how to publish a new version of the library.

Copy link
Copy Markdown
Contributor Author

kieran-osgood-shopify commented Mar 25, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

Upgrade sample apps to Apollo v2 and re-run code generation. Apollo v1 generated code caused Swift 6 concurrency issues, so the generated GraphQL fragments and operations were regenerated with the v2 generator (formatting/structure changes only). The Xcode project references were updated to remove obsolete generated entries and align with the new code layout. This change removes the temporary workaround and enables the 2025-07 Swift upgrade / follow-up PR to proceed.
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.

1 participant