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

Field Merging [4/x] Disable SelectionSetInitializers when field merging is enabled #343

Conversation

AnthonyMDev
Copy link
Contributor

@AnthonyMDev AnthonyMDev commented Apr 24, 2024

These options are incompatible and cannot be used together. Validation to prevent this is added in next PR.

@AnthonyMDev AnthonyMDev changed the title Field Merging [4/x] Fixing missing type references Field Merging [4/x] Disable SelectionSetInitializers when field merging is enabled Apr 24, 2024
@AnthonyMDev AnthonyMDev marked this pull request as ready for review April 24, 2024 19:51
@AnthonyMDev AnthonyMDev force-pushed the Implement_FieldMerging_in_SelectionSetTemplate branch from 524a720 to 5bddba3 Compare May 1, 2024 01:09
@AnthonyMDev AnthonyMDev force-pushed the Field_Merging_4_x_disable_initializers branch from 96528f0 to b58915d Compare May 1, 2024 01:09
@AnthonyMDev AnthonyMDev force-pushed the Implement_FieldMerging_in_SelectionSetTemplate branch from 5bddba3 to 288253c Compare May 1, 2024 01:20
@AnthonyMDev AnthonyMDev force-pushed the Field_Merging_4_x_disable_initializers branch from b58915d to fd51cc3 Compare May 1, 2024 01:20
@AnthonyMDev AnthonyMDev force-pushed the Implement_FieldMerging_in_SelectionSetTemplate branch from 288253c to 391fa0d Compare July 16, 2024 18:36
@AnthonyMDev AnthonyMDev force-pushed the Field_Merging_4_x_disable_initializers branch from fd51cc3 to ce27eab Compare July 16, 2024 18:39
@AnthonyMDev AnthonyMDev force-pushed the Implement_FieldMerging_in_SelectionSetTemplate branch from 391fa0d to a3e3b18 Compare July 16, 2024 19:51
@AnthonyMDev AnthonyMDev force-pushed the Field_Merging_4_x_disable_initializers branch from ce27eab to af2c194 Compare July 16, 2024 19:51
@AnthonyMDev
Copy link
Contributor Author

Closed in favor of #431

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