bug: GraphiQL Doc Explorer Loading State about explore.opensauced.pizza HOT 5 OPEN

Adding this here for potentially easier resolution: the fetcher function takes care of placing schema property here - a working fix should be removing that line entirely and separating the GraphiQL fetcher from our config.fetchOneGraph - this could be potentially done in isolation, removing this fix bleeding into the other 2 issues reported, however closely linked IMO

from explore.opensauced.pizza.

So far just removing the line from Explorer component actually breaks things. Passing in undefined for initial schema doesn't help either. Though I'm seeing it in some of the source files like we linked above, looks like the spinner state doesn't make it into the node_modules/graphiql/dist or node_modules/graphiql/esm files somehow.

from explore.opensauced.pizza.

So far just removing the line from Explorer component actually breaks things. Passing in undefined for initial schema doesn't help either. Though I'm seeing it in some of the source files like we linked above, looks like the spinner state doesn't make it into the node_modules/graphiql/dist or node_modules/graphiql/esm files somehow.

we need to stop polluting the schema property too unless we create our wrapper function - works for me locally, maybe you are seeing a different bug ?

from explore.opensauced.pizza.

Screenshots or it didn’t happen ;-) with dead ends I hit I tossed my changes so if you’ve got a fix 🚢

from explore.opensauced.pizza.

just went to the version before we adjusted the schema, removed the schema prop entirely AND left the default fetcher. That is the most basic example of GraphiQL with just the fetcher added - can't not not work if you get what I mean :D

from explore.opensauced.pizza.