module page by ramseycoder · Pull Request #92 · nan-ci/platform

ramseycoder · 2021-10-18T19:02:53Z

make some module page

linear · 2021-10-18T19:02:54Z

NAN-21 Page de module

View original card in Trello

Kouamé Rameaux Koffi 2021-10-13

pull request : #80
nouveau build : https://bca1170a.platform-nan-dev-8sl.pages.dev/module

Kouamé Rameaux Koffi 2021-10-08

pull request : #80
lien build : https://487a958c.platform-nan-dev-8sl.pages.dev/module

cloudflare-workers-and-pages · 2021-10-18T19:04:40Z

Deploying with Cloudflare Pages

Latest commit:	`6e69893`
Status:	✅ Deploy successful!
Preview URL:	https://0b8d1eba.platform-nan-dev-8sl.pages.dev

View logs

kigiri · 2021-10-18T19:05:43Z

app.jsx

 import { Router } from './lib/router.js'
 import { Profile } from './page/profile.jsx'
 import { Home } from './page/home.jsx'
+import {Module} from './page/module.jsx'


il manque prettier la

kigiri · 2021-10-18T19:06:30Z

component/stackLi.jsx

+export const StackLi = ({ data: { type, name } }) => {
+  return (
+    <li class="stackli">
+      <Span class={`${type === 'exercise' ? 'square' : 'circle'}`} />


pourqoui creer une string c'est deja des strings, fait plutot:

<Span class={type === 'exercise' ? 'square' : 'circle'} />

kigiri · 2021-10-18T19:09:59Z

page/module.jsx

+      { name: '06-Duplicate', type: 'exercise' },
+      { name: '07-cool', type: 'quiz' },
+    ],
+  },


Alors je suis desoler mais une partie de mes remarques n'avais pas ete publier:

#80 (review)
si tu peu recheck, notament niveau de la structure

kigiri

Il faut que tu formatte ta donner pour qu'elle soit adapter a ton utilisation, la tu la manipule de partout c'est pas optimal

kigiri · 2021-10-18T23:16:55Z

data/fakeModule.json

+    {
+      "name": "02-Anything To Declare",
+      "type": "exercise",
+      "key": "02-Anything To Declare"


bon c'est pas important pour ce code la, mais en toute logique la key correspondra au nom du fichier, alors que le name lui sera le titre (H1) du markdown du sujet

kigiri · 2021-10-18T23:24:09Z

page/module.jsx

+              data={rest}
+              isPassed={isPassed(rest.name)}
+              isNext={isNext(rest.name)}
+              infos={


infos est aussi imprecis que data comme nom, a eviter le plus possible.

kigiri · 2021-10-18T23:30:13Z

page/module.jsx

+          {fakeModule.children.map(({ key, ...rest }) => (
+            <StackLi
+              key={key}
+              data={rest}


data... j'ai pas deja fait une remarque sur ce genre de nom de proprieter / variable ?

ca dit rien ! c'est quoi comme data ?
pourquoi ne pas juste spread les props:

fakeModule.children.map((child) => ( <StackLi {...child} result={userInfo.results[child.key]} /> ))

kigiri · 2021-10-18T23:31:33Z

page/module.jsx

        <ul class="stack">
-          {fakeModule.configs.display.map((s) => (
-            <StackLi key={s.name} data={s} />
+          {fakeModule.children.map(({ key, ...rest }) => (


je comprend pas pourquoi t'a besoin du isNext, et pas sur du nom de la variable tu veut dire que c'est le prochain qui est possible a faire ? ou la tache en cours ?
Dans ce cas ca serais plus current ou isActive

kigiri · 2021-10-18T23:32:04Z

page/module.jsx

+  const isNext = (name) => {
+    const [module, step] = userInfo.level
+      .split('-')
+      .map((r) => parseInt(r.slice(1)))


C'est mieu d'utiliser Number que parseInt

kigiri · 2021-10-18T23:43:17Z

page/module.jsx

+        userProgress.find((r) => getExoNumber(r[0]) === exoNumber)[1].pass)
+    ) {
+      return true
+    } else return false


Ce genre de code:

if ( module > moduleNumber || (module === moduleNumber && step >= exoNumber && userProgress.find((r) => getExoNumber(r[0]) === exoNumber)[1].pass) ) { return true } else return false

Ca va pas, ta condition est trop complexe, decompose la:

// est-ce qu'on devrais ce fier a ca ? if (module < moduleNumber) return true // trying to peek ? if (module > moduleNumber || step < exoNumber) return false return userProgress.find((r) => getExoNumber(r[0]) === exoNumber)[1]?.pass

Mais toute cette fonction ne sert a rien

kigiri · 2021-10-19T13:37:42Z

data/fakeModule.json

-      "name": "05-Redeclaration Love",
+      "name": "Redeclaration Love",
      "type": "exercise",
      "key": "05-Redeclaration of Love"


pas tres important mais les keys seront pas genre: 05-Redeclaration of Love mais le nom du fichier donc plus un truc genre: 05-redeclaration-of-love.exercise.md je pense

kigiri · 2021-10-19T13:40:06Z

page/module.jsx

-                  : null
-              }
+              {...child}
+              name={child.key}


pourquoi name={child.key} ? alors que le child a deja un name ?

kigiri · 2021-10-19T13:40:55Z

component/stackLi.jsx

  const iconType = {
    quiz: '📚',
    exercise: '🖊',
  }


le donne statique comme ca vaut mieu les sortir que les redeclarer a chaques fois, ca changes pas grand chose mais c'est toujours preferable

kigiri · 2021-10-19T14:10:00Z

component/stackLi.jsx

+  }
+
+ .module .stackli span.name  {
+   font-size: 1.2rem;


j'ai fait un commentaire sur changer la font-size et les margin, l'idee c'est que l'affichage corresponde a ce qu'un editeur de texte peu generer.

kigiri · 2021-10-25T16:56:53Z

component/stackLi.jsx

+        {result && <> &nbsp;<Div class="second_block">
+            {' '}
+            (<Span class="attempts">{result.attempts}-attempts</Span>
+            {result.seconds && <>&nbsp;


<>  😨
t'a passer prettier la dessus ?

kigiri · 2021-10-25T16:59:41Z

component/markdown.jsx

+        children,
+      ),
+  ]),
+)


cool mais gere l'espacement avec un padding ou une marge plutot que &nbsp, tu peu indiquer 1ch pour avoir la bonne valeur

add : module page

000bb7d

kigiri requested changes Oct 18, 2021

View reviewed changes

fix: review correction

e1198f4

kigiri requested changes Oct 18, 2021

View reviewed changes

ramseycoder self-assigned this Oct 19, 2021

kigiri reviewed Oct 19, 2021

View reviewed changes

update: add review correction

ebf26da

ramseycoder force-pushed the ramsey/nan-21-page-de-module branch from 148be57 to ebf26da Compare October 19, 2021 14:55

ramseycoder added 2 commits October 19, 2021 15:28

fix: remove margin and font-size

f388445

fix: update style

ea491a0

kigiri reviewed Oct 25, 2021

View reviewed changes

fix: adding margin and padding instead of  

6e69893

Comments

Conversation

ramseycoder commented Oct 18, 2021

Uh oh!

linear bot commented Oct 18, 2021

Uh oh!

cloudflare-workers-and-pages bot commented Oct 18, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying with Cloudflare Pages

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kigiri left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kigiri Oct 25, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

cloudflare-workers-and-pages bot commented Oct 18, 2021 •

edited

Loading

kigiri Oct 25, 2021 •

edited

Loading