-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Closed
Labels
graphiqlproposalrfcRequest for CommentRequest for Commentwork-in-progressnot ready to merge yet, PR is available for demonstration/review purposesnot ready to merge yet, PR is available for demonstration/review purposes
Description
As per the discussion at this week's working group meeting, I'll be putting together a proposal for replacing the use of React Context with an alternative state manager. Goals for this RFC are to reduce component complexity in the graphiql-react
package and to unlock the possibility to more easily provide GraphQL IDE building blocks for non-React frameworks.
Here are the tasks that I'd like to accomplish to make this a successful RFC:
- Add a Ladle installation as a component development environment
- Pull
explorer
plugin out ofgraphiql-react
and into a separate plugin package- rewrite context -> zustand
- refactor tests
- Pull
history
plugin out ofgraphiql-react
and into a separate plugin package- rewrite context -> zustand
- refactor tests
- Add a new
store
folder tographiql-toolkit
that contains state slices that match up with existing contexts - storage
- editor
- schema
- execution
- plugin
acao and baohouse
Metadata
Metadata
Assignees
Labels
graphiqlproposalrfcRequest for CommentRequest for Commentwork-in-progressnot ready to merge yet, PR is available for demonstration/review purposesnot ready to merge yet, PR is available for demonstration/review purposes