Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Defer Selection Set Template - Basic (W/clean up) #118

Merged

Conversation

AnthonyMDev
Copy link
Contributor

This is an alternative to #113. It's all the same code behaviors and tests are the same. I just took that PR and improved the usage of TemplateString in the SelectionSetTemplate.

@calvincestari calvincestari force-pushed the defer/selection-set-template-basic-anthony branch from 9e7848d to 6cb8b16 Compare November 1, 2023 20:21
Copy link
Member

@calvincestari calvincestari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I've rebased this after merging #116 and fixed up the selection set template tests. Should be good for merge now.

@calvincestari calvincestari merged commit de9e35b into feature/defer Nov 1, 2023
12 checks passed
@calvincestari calvincestari deleted the defer/selection-set-template-basic-anthony branch November 1, 2023 20:32
BobaFetters pushed a commit that referenced this pull request Nov 2, 2023
f499efa4 chore: Disable `@defer` support (#122)
70fd68cd Merge branch 'main' into feature/defer
d6c49617 Defer Selection Set Template - Basic (W/clean up) (#118)
f5dc1758 merge: `feature/defer` update to `main` (#116)
2c5463b0 refactor: [1/x] Refactor root field builder for deferred fragments (#74)
25d44b83 Merge branch 'main' into feature/defer
1d8424d3 Merge branch 'main' into feature/defer
a8c31ca3 Merge branch 'main' into feature/defer
d1a3c0ab feature: Operation definition template supports the `@defer` directive (#31)
6a01891b Merge branch 'project-breakup' into feature/defer
11cf5828 Merge branch 'project-breakup' into feature/defer
723b8648 refactor: Don't require `label` argument for deferred named fragments (#33)
9041eee0 feature: Require `label` argument on `@defer` directive (#28)
01b81541 Migrating defer branch to new project structure

git-subtree-dir: apollo-ios-codegen
git-subtree-split: f499efa4ee99c7eb86e5f2e6d0b630756c48f0de
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.

2 participants