Skip to content
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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

maximecolin
Copy link
Member

@maximecolin maximecolin commented Sep 5, 2024

Copy link
Contributor

@xavier-rdo xavier-rdo left a 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 🙏

content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
@xavier-rdo
Copy link
Contributor

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 ?

@maximecolin
Copy link
Member Author

maximecolin commented Sep 6, 2024

@xavier-rdo

Est-ce qu'il ne faudrait pas également dépublier l'ancienne étude de cas MJC si ça n'est pas déjà fait ?

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.

@maximecolin
Copy link
Member Author

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.

github-actions bot pushed a commit that referenced this pull request Sep 6, 2024
github-actions bot pushed a commit that referenced this pull request Sep 6, 2024
github-actions bot pushed a commit that referenced this pull request Sep 6, 2024
github-actions bot pushed a commit that referenced this pull request Sep 6, 2024
content/case-study/mjc-marque-blanche.md Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved

#### 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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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.

content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Show resolved Hide resolved
Copy link
Contributor

@quentin-brouillet quentin-brouillet left a 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

content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Show resolved Hide resolved
@ogizanagi
Copy link
Member

ogizanagi commented Sep 6, 2024

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.

Tu peux ajouter show: false aux frontmatter des termes pour lesquels tu ne souhaites pas référencer de page dédiée

/** Show a dedicated page or not */
public bool $show = true;

github-actions bot pushed a commit that referenced this pull request Sep 10, 2024
github-actions bot pushed a commit that referenced this pull request Sep 10, 2024
github-actions bot pushed a commit that referenced this pull request Sep 10, 2024
content/term/pwa.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
content/case-study/mjc-marque-blanche.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants