New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

✨ using graphql + ssr #4

Merged

fabienjuif merged 15 commits into cv from cv-graphql-ssr

Nov 19, 2018

Member

fabienjuif commented Nov 5, 2018 •

edited

Loading

Supprimer le Loading .... qui est rendu au début
Pour ça bien lire ça : https://www.apollographql.com/docs/react/features/server-side-rendering.html
Découper en n requête (plutôt qu'une grosse sur CV)
Faire un HoC pour rendre le tout plus propre
Routes
REproduire le bogue entre react / apollo / styled-component pour le ssr


          ✨ using graphql + ssr

d951802

fabienjuif added the invalid label

fabienjuif commented

View reviewed changes

packages/api-gateway/src/graphql.js Outdated Show resolved Hide resolved

fabienjuif added 3 commits

November 5, 2018 17:08


          ✨ seems working... have to clean and reproduce bug

0ccd370


          🔨 skills format

376e553


          🚨 fix templating / data / ssr

5d6f8a8

fabienjuif commented

View reviewed changes

packages/api-gateway/src/react.js Outdated Show resolved Hide resolved

fabienjuif commented

View reviewed changes

packages/api-gateway/package.json Outdated Show resolved Hide resolved

fabienjuif commented

View reviewed changes

packages/ui/package.json Outdated Show resolved Hide resolved

fabienjuif commented

View reviewed changes

packages/ui/src/addNewProposal.js Show resolved Hide resolved

fabienjuif commented

View reviewed changes

packages/ui/src/app.js Outdated Show resolved Hide resolved

fabienjuif commented

View reviewed changes

packages/ui/src/client.js Outdated Show resolved Hide resolved

fabienjuif commented

View reviewed changes

packages/ui/src/cv/experiences/experience.js Show resolved Hide resolved

fabienjuif commented

View reviewed changes

packages/ui/src/cv/who/who.js Show resolved Hide resolved

fabienjuif commented

View reviewed changes

packages/ui/src/index.js Outdated Show resolved Hide resolved

fabienjuif added 8 commits

November 5, 2018 17:48


          🎨 cleanup Who

4e8f8aa


          🔥 isomorphic-fetch

f1a6915


          ⬆️ some packages

76c2cff


          ♻️ hardpaste normalize.css into index.html avoiding clipping

6a548b2


          ✨ query variable

08ff212


          🎨 split queries

6d5eec8


          ✨ ssr cache

32ef6a3


          🎨 hoc (#6)

457bfd2

fabienjuif mentioned this pull request

graphql / default values / mutate from null to undefined ? #10

Open


          🎨 eslint

053e55f

fabienjuif mentioned this pull request

try to move back to .jsx files #8

Closed

fabienjuif added 2 commits

November 19, 2018 15:02


          🎨 more lint

0c1f262


          ✨ router

75eedb7

fabienjuif merged commit 81cb526 into cv

fabienjuif deleted the cv-graphql-ssr branch

November 19, 2018 14:21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid