relay icon indicating copy to clipboard operation
relay copied to clipboard

Published 20 hours ago • verified publisher icon graphql-dotnet

Is 'clientMutationId' needed anymore?

Open fiyazbinhasan opened this issue 3 years ago • 1 comments

As of this pull request, https://github.com/facebook/relay/commit/3838691f4f815ce90b6d544fa052b112259a4c9a

RelayModern has no requirements on mutations beyond the general mutation specs. Specifically, the additional requirements that are no longer required are:

  • clientMutationId (even RelayClassic only required this because of FB internal reasons)
  • $input objects, with RelayModern the input to mutations can come either as one big object or multiple mutation arguments.

As clientMutationId is no longer needed in RelayModern, we can remove the MutationInputGraphType altogether. Instead, we can use the actual InputGraphType. Do you guys agree?

fiyazbinhasan avatar Apr 28 '21 17:04 fiyazbinhasan