Skip to content

Commit 3ef0167

Browse files
Update TUTORIAL.md
Co-authored-by: Scott Walkinshaw <scott.walkinshaw@gmail.com>
1 parent 91e0ca9 commit 3ef0167

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

TUTORIAL.md

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -951,13 +951,12 @@ input CollectionInput {
951951
*Rule #21: Structure mutation inputs to reduce duplication, even if this
952952
requires relaxing requiredness constraints on certain fields.*
953953

954-
As in the example above, `collectionUpdate` takes two arguments. The first
955-
selects the collection to update, and the second provides the update data. An
956-
alternative to this would be one argument, `CollectionInput!`, which would
957-
include the ID. In fact, [some older Shopify APIs follow this
958-
pattern](https://shopify.dev/docs/api/admin-graphql/2023-07/mutations/productUpdate).
959-
However, this is no longer recommended, as it makes it difficult to determine
960-
which parts of the call relate to 'select', and which relate to 'update'.
954+
As in the example above, `collectionUpdate` takes two arguments. `collectionId`
955+
selects the collection to update, and `collection` provides the update data. An
956+
alternative to this would be a single merged `collection: CollectionInput!` argument, where
957+
`CollectionInput` would have a nullable `id` input field. However, this makes it
958+
difficult to determine which parts of the call relate to 'select', and which relate
959+
to 'update' and is not recommended.
961960

962961
*Rule #22: For update mutations, the parameter relating to selecting the entry
963962
must be separate to the parameter providing the change data.*

0 commit comments

Comments
 (0)