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

fix: Removes local cache mutation type condition setter #485

Merged
merged 2 commits into from
Sep 18, 2024

Conversation

calvincestari
Copy link
Member

Fixes apollographql/apollo-ios#3443.

Simply removes the setter for mutable inline fragments (type conditions). The correct way to initialize with the type condition is to use asRootEntityType.

Copy link
Contributor

@AnthonyMDev AnthonyMDev left a comment

Choose a reason for hiding this comment

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

Perfect. Nice and easy fix!

@calvincestari calvincestari force-pushed the fix/codegen-spm-generated-swift-tools-version branch from ff4a7a6 to 2fc5a2f Compare September 18, 2024 17:01
Base automatically changed from fix/codegen-spm-generated-swift-tools-version to main September 18, 2024 17:15
@calvincestari calvincestari force-pushed the fix/local-cache-mutation-type-condition-setter branch from 9c991b3 to 8a71a6e Compare September 18, 2024 17:19
Copy link

netlify bot commented Sep 18, 2024

Deploy Preview for apollo-ios-docc canceled.

Name Link
🔨 Latest commit 8a71a6e
🔍 Latest deploy log https://app.netlify.com/sites/apollo-ios-docc/deploys/66eb0ba97bd3d40008b01769

Copy link

netlify bot commented Sep 18, 2024

Deploy Preview for eclectic-pie-88a2ba canceled.

Name Link
🔨 Latest commit 8a71a6e
🔍 Latest deploy log https://app.netlify.com/sites/eclectic-pie-88a2ba/deploys/66eb0ba954c5600008702a77

@calvincestari calvincestari merged commit 1611193 into main Sep 18, 2024
37 checks passed
@calvincestari calvincestari deleted the fix/local-cache-mutation-type-condition-setter branch September 18, 2024 17:29
BobaFetters pushed a commit to apollographql/apollo-ios-codegen that referenced this pull request Sep 18, 2024
BobaFetters pushed a commit that referenced this pull request Sep 18, 2024
cba4dd7d fix: Removes local cache mutation type condition setter (#485)

git-subtree-dir: apollo-ios-codegen
git-subtree-split: cba4dd7d00604092950a9e340ef318e74287ce03
BobaFetters pushed a commit that referenced this pull request Sep 18, 2024
…on type condition setter

git-subtree-dir: apollo-ios-codegen
git-subtree-mainline: 53f7cad
git-subtree-split: cba4dd7d00604092950a9e340ef318e74287ce03
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.

Remove local cache mutation type case setter
2 participants