Does 'to_dict' will be implemented in graphql-core-next?

[yanghg-basefx]

I raise an issue before Out of memory when wrong use graphql with dataloader, but now I find this repo and to_dict is not exist in ExecutionResult. Does it will be implemented? If not, please close that issue, and I will remove it in my project.

But GraphQL-Server is using this function, please note.

Thanks so much!

[Cito]

This just returns a dictionary with error and data as keys (instead of the named tuple), and it formats the errors. I don't find this really useful, and it does two things at once which has always a certain smell.

I'd like to get input from @syrusakbary - can this be changed in GraphQL-Server to make it compatible? Or should I really add this to graphql-core-next?

[Cito]

There will be a new version of GraphQL-server-core based on GraphQL-core-next (graphql-python/graphql-server#20), which will probably not need this, or maybe we will add it then if necessary/reasonable.