Skip to content

Commit b55e497

Browse files
benjiebinaryseed
authored andcommitted
InputUnion: tweak to plain data (#655)
* InputUnion: tweak to plain data * Update overview also
1 parent a0cd873 commit b55e497

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

rfcs/InputUnion.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ Clients should be able to pass "natural" input data to unions without specially
270270

271271
| [1](#-1-explicit-__typename-discriminator-field) | [2](#-2-explicit-configurable-discriminator-field) | [3](#-3-order-based-discrimination) | [4](#-4-structural-uniqueness) | [5](#-5-one-of-tagged-union) |
272272
|----|----|----|----|----|
273-
| ⚠️ | ⚠️ ||| 🚫 |
273+
| ⚠️ | ⚠️ ||| ⚠️ |
274274

275275
## 🎯 I. Input unions should be easy to upgrade from existing solutions
276276

@@ -744,7 +744,7 @@ type Mutation {
744744
* [F. Migrating a field to a polymorphic input type is non-breaking](#-f-migrating-a-field-to-a-polymorphic-input-type-is-non-breaking)
745745
* ✅ No migration required, as this pattern is already possible
746746
* [H. Input unions should accept plain data](#-h-input-unions-should-accept-plain-data)
747-
* 🚫 The shape of a data structure is forced to contain an intermediate type
747+
* ⚠️ The data is wrapped in a (simple) container type
748748
* [I. Input unions should be easy to upgrade from existing solutions](#-i-input-unions-should-be-easy-to-upgrade-from-existing-solutions)
749749
* ✅ No migration required, as this pattern is already possible
750750
* [J. A GraphQL schema that supports input unions can be queried by older GraphQL clients](#-j-a-graphql-schema-that-supports-input-unions-can-be-queried-by-older-graphql-clients)
@@ -764,7 +764,7 @@ A quick glance at the evaluation results. Remember that passing or failing a spe
764764
| [E](#-e-a-member-type-may-be-a-leaf-type) | 🚫 | 🚫 | ✅⚠️ | 🚫 ||
765765
| [F](#-f-migrating-a-field-to-a-polymorphic-input-type-is-non-breaking) | ✅⚠️ | ✅⚠️ || ⚠️ ||
766766
| [G](#-g-input-unions-may-include-other-input-unions) ||||||
767-
| [H](#-h-input-unions-should-accept-plain-data) | ⚠️ | ⚠️ ||| 🚫 |
767+
| [H](#-h-input-unions-should-accept-plain-data) | ⚠️ | ⚠️ ||| ⚠️ |
768768
| [I](#-i-input-unions-should-be-easy-to-upgrade-from-existing-solutions) | ✅⚠️ | ✅⚠️ || ⚠️ ||
769769
| [J](#-j-a-graphql-schema-that-supports-input-unions-can-be-queried-by-older-graphql-clients) ||||||
770770
| [K](#-k-input-unions-should-be-expressed-efficiently-in-the-query-and-on-the-wire) ||||||

0 commit comments

Comments
 (0)