No data in `GraphQL::Pro::Defer::Deferral::to_h` when errors are present

[boazreicher]

The to_h method of GraphQL::Pro::Defer::Deferral does not return any data when errors are present.

This seems to contradict the docs which say .to_h returns a hash with path:, data:, and/or errors: (tbf, the comment in the method signature is more clear/explicit: @return [Hash] with 'path:', either 'errors:' or 'data:', and 'label:', if given.)

It seems like it would be more useful to return both errors and data, and let the client code decide what to do with them.

[rmosolgo]

Hey, thanks for the suggestion. I'm not exactly sure why it was originally implemented that way ... @defer and @stream have always been in the "proposal" stage, but it looks like the current proposal (graphql/graphql-spec#1110 ?) does allow for both errors and data. I can't think of why it would be bad to include both. I'll give it a try and follow up here.

[rmosolgo]

👋 I just released graphql-pro v1.29.10 which includes "data" in the payload even if there are "errors". Please give it a try and let me know what you think!