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

feat: page validateur bal #1834

Merged
merged 6 commits into from
Oct 2, 2024
Merged

feat: page validateur bal #1834

merged 6 commits into from
Oct 2, 2024

Conversation

MaGOs92
Copy link
Contributor

@MaGOs92 MaGOs92 commented Sep 24, 2024

Ajoute le validateur BAL

@MaGOs92 MaGOs92 requested a review from vinsag September 24, 2024 11:46
@MaGOs92 MaGOs92 force-pushed the gfay_feat_validateur-bal branch from d10b80c to 7f9d60b Compare September 25, 2024 12:28
@MaGOs92 MaGOs92 marked this pull request as ready for review September 25, 2024 12:28
@MaGOs92 MaGOs92 force-pushed the gfay_feat_validateur-bal branch 2 times, most recently from 63e3e14 to 844f354 Compare September 26, 2024 08:11
@vinsag vinsag requested a review from nkokla September 30, 2024 07:33
@@ -35,6 +38,7 @@
"remark": "^15.0.1",
"remark-heading-id": "^1.0.1",
"remark-html": "^16.0.1",
"sharp": "^0.33.5",
Copy link
Contributor

Choose a reason for hiding this comment

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

Hello @MaGOs92 !
Pourquoi a t'on besoin de sharp ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Salut Nicolas, c'est une recommandation que j'ai vu dans les logs de build de Scalingo

Copy link
Contributor

Choose a reason for hiding this comment

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

C'est très étrange. Le log precise dans quel cas il a besoin de cette dependances ?
Au pire, est-ce qu'il ne devrait pas être en devDependencies ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Voici le log en question :
Capture d’écran 2024-10-01 à 16 58 40

Copy link
Contributor Author

Choose a reason for hiding this comment

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

En faite c'est pas au build mais au runtime qu'apparait ce log

Copy link
Member

Choose a reason for hiding this comment

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

En fait il faut installer la dépendance sharp non présent dans next.js : https://nextjs.org/docs/messages/sharp-missing-in-production

@MaGOs92 MaGOs92 force-pushed the gfay_feat_validateur-bal branch from c6f5af3 to e928056 Compare October 1, 2024 15:03
@MaGOs92 MaGOs92 force-pushed the gfay_feat_validateur-bal branch from 1149548 to 9bcae0b Compare October 2, 2024 12:04
@MaGOs92 MaGOs92 merged commit b8f943e into main Oct 2, 2024
4 checks passed
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.

3 participants