-
Notifications
You must be signed in to change notification settings - Fork 11
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.
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
Introduce MJC White Label case study #628
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci Maxime et Eva ! Quelques remarques, suggestions et corrections d'orthographe/grammaire, sinon 👍
Remarque : ça reste des suggestions, libre à vous de les adopter ou non 🙏
Remarque : le déploiement en staging n'a pas fonctionné (sans doute à cause de la CI HS ?) Est-ce qu'il ne faudrait pas également dépublier l'ancienne étude de cas MJC si ça n'est pas déjà fait ? |
6d01b04
to
3c4d0e4
Compare
L'ancienne étude de cas me semble toujours pertinente, elle décrit un travail différent de celui fait sur la marque blanche et les fonctionnels décrit dedans sont toujours existant. |
J'ai du rajouter deux pages sur les termes Next.js et PWA qui n'existait pas alors que les termes étaient référencés dans l'étude de cas, ça bloquait le build. Dans un soucis de gain de temps, j'ai mis du blabla ChatGPT, en vérifiant bien sur qu'il racontait pas n'importe quoi. |
5b02c38
to
f401653
Compare
|
||
#### Seconde étape | ||
|
||
La seconde étape a été de développer les deux applications front et admin en React. Ces deux applications communiquaient avec le métier via la nouvelle API évoquée plus haut. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
La seconde étape a été de développer les deux applications front et admin en React. Ces deux applications communiquaient avec le métier via la nouvelle API évoquée plus haut. | |
La seconde étape a été de développer les deux applications **front** et **admin** en React. Ces deux applications communiquaient avec le métier via la nouvelle API évoquée plus haut. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci Max, qq pétouilles ici et là mais l'essentiel est bien retranscris
Tu peux ajouter elao_/src/Model/Glossary/Term.php Lines 20 to 21 in f0219a5
|
for commit df4b7b5
for commit df4b7b5
7fc757e
to
f4acb8f
Compare
f4acb8f
to
6f01a42
Compare
for commit 4ce5bfa
for commit 91b53db
https://elao.github.io/elao_/pr/628/etudes-de-cas/mjc-marque-blanche