wundergraph / cosmo

The open-source solution to building, maintaining, and collaborating on GraphQL Federation at Scale. The alternative to Apollo Studio and GraphOS.
https://cosmo-docs.wundergraph.com/
Apache License 2.0
707 stars 102 forks source link

New server response format when not 200 code in playground looks unreadable #1185

Open flymedllva opened 1 month ago

flymedllva commented 1 month ago

Component(s)

studio

Is your feature request related to a problem? Please describe.

I extremely dislike the way it now shows responses not having a 200 code

Screenshot 2024-09-18 at 12 24 37
  1. it's unreadable, although the idea to show what came over HTTP is understandable, but it's better to add some custom fields to the standard GraphQL response structure. Now I don't understand how to try to read responseBody during debugging, and other fields don't give useful load
  2. We subgraphs return 4 error codes in case of non-significant problems, 5 in case of unknown server errors. At the same time they describe the standard GraphQL response structure, deviations from this logic are possible only in case of some serious problems, but it seems Playground should be used mainly for debugging, why should I deviate from this structure trying to invent another one?

Describe the solution you'd like

Describe alternatives you've considered

No response

Additional context

No response

github-actions[bot] commented 1 month ago

WunderGraph commits fully to Open Source and we want to make sure that we can help you as fast as possible. The roadmap is driven by our customers and we have to prioritize issues that are important to them. You can influence the priority by becoming a customer. Please contact us here.

thisisnithin commented 1 month ago

Hi @flymedllva . As we cover more use cases we update the ui accordingly. We have an idea on how this can be improved. Thanks for the insight.

flymedllva commented 1 week ago

Hi! @thisisnithin Is there no information on what this will look like in the future? Is it possible at least for 4** response codes not to change the GraphQL structure of the response?

thisisnithin commented 4 days ago

Hi, this is in progress as part of a bigger PR