search

graphql-kit / graphql-voyager

🛰️ Represent any GraphQL API as an interactive graph
https://graphql-kit.com/graphql-voyager/
MIT License
7.81k stars 516 forks source link

Dependency upgrades and new simplified demo #228

Closed LunaticMuch closed 2 years ago

LunaticMuch commented 2 years ago

This is a combo PR that includes the following:

  1. Upgrade of various dependencies
  2. A brand new demo which is simplified and does not include the whole website package
  3. Material UI v5

I moved the version to 1.1.0 - whilst the versioning can be debatable, I believe we're well out from a release candidate and move to semantic versioning from here.

LunaticMuch commented 2 years ago

Feels like it does not build correctly after the new merge from the upstream 😞

IvanGoncharov commented 2 years ago

A brand new demo which is simplified and does not include the whole website package

@LunaticMuch I don't see a new demo only an old one removed.

LunaticMuch commented 2 years ago

@LunaticMuch I don't see a new demo only an old one removed.

There's just an index which uses voyager as a lib against a public endpoint.

Reckon this branch is now in trouble because we made internally all upgrades which are now coming to master plus the demo thing.

It was initially complicated to extract various piece, so I made a combo PR which I knew was not a good approach....

I can resubmit two separate PRs, one for demo with the index only if you like this, and one for material ui v5. But I need to wait for all other deps upgrades you're sending in, otherwise we keep having conflicts.

IvanGoncharov commented 2 years ago

Material UI v5

Can you pleae extract this one into separate PR? It is super useful and should be an easy merge.

LunaticMuch commented 2 years ago

@IvanGoncharov is doing already deps.

229 is for material UI v5.

Anything else will be in other PRs