Closed moreal closed 1 year ago
I think it is a good idea to have a common GraphQL API interface between applications using libplanet to allow for interop, however as it seems for planet-node for the moment it seems duplicated with queries in PlanetNodeSchema, so I'm wondering if we should deduplicate the code. Any thoughts? @dahlia @longfin
It's ideal to remove the most of PlanetNodeSchema
and use LibplanetExplorerSchema<T>
, which is common, instead. Also it would be better if PlanetNodeSchema
at the root composites LibplanetExplorerSchema<T>
(or LibplanetExplorerSchema<T>
is served at the root, but it's just idealism) rather than LibplanetExplorerSchema<T>
having a separated endpoint /explorer.
Also it would be better if
PlanetNodeSchema
at the root compositesLibplanetExplorerSchema<T>
(orLibplanetExplorerSchema<T>
is served at the root, but it's just idealism) rather thanLibplanetExplorerSchema<T>
having a separated endpoint /explorer.
After looking at your idea, I felt it seems similar to GraphQL federation (https://github.com/graphql-dotnet/graphql-dotnet/pull/3144). But it seems different a little too.
It's ideal to remove the most of PlanetNodeSchema and use LibplanetExplorerSchema
, which is common, instead.
At this time, I'll remove the ExplorerQuery<T>
part in PlanetNodeSchema
. (0a31ade)
Could you review this pull request one more time, please? Is there something I have to work on or another opinion?
This PR has 7
quantified lines of changes. In general, a change size of upto 200
lines is ideal for the best PR experience!
Was this comment helpful? :thumbsup: :ok_hand: :thumbsdown: (Email) Customize PullRequestQuantifier for this repository.
This pull request close #41.
This pull request makes planet-node provide
LibplanetExplorerSchema<T>
on the/explorer
endpoint.I want to suggest all libplanet implementations reuse
LibplanetExplorerSchema<T>
and planetarium/libplanet-explorer-frontend should target the schema for generic support.Because I don't have permission to assign reviewers, I mention here:
Related issues: