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

Implémentation de Alise #466

Merged
merged 4 commits into from
Dec 17, 2024

Conversation

raphckrman
Copy link
Contributor

@raphckrman raphckrman commented Dec 15, 2024

🚀 Nouvelle Pull Request

Proposez vos modifications pour améliorer Papillon

Informations importantes

Merci de vous référer à la documentation sur la contribution si vous avez des questions à propos des pull requests (https://gitbook.getpapillon.xyz/organisation/outils-internes/github)

Checklist d'avant pull request

Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].

  • Vous avez testé de build le projet avec vos modifications et ce build a réussi
  • Vous respectez les conventions de codage et de nommage du projet
  • Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
  • Cette pull request n'est pas un duplicata d'une autre
  • Cette pull request est prête à être revue (review) et fusionnée (merge)
  • Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
  • Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
  • Les détails des changements ont été décrits ci-dessous
  • Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

Implémentation de Alise, merci à FuNnIXx de m'avoir prêté son compte afin de tester

Preview

Configuration de la cantine Connexion Page principale Historique
image image image image

@raphckrman
Copy link
Contributor Author

⚠️ Je viens une nouvelle fois de fix le Lint, merci de faire attention dans les push/PR car après c'est chiant pour le workflow

@Kgeek33
Copy link
Contributor

Kgeek33 commented Dec 15, 2024

⚠️ Je viens une nouvelle fois de fix le Lint, merci de faire attention dans les push/PR car après c'est chiant pour le workflow

Tkt je l'ai amélioré le workflow de Typescript, voir => #449
Ça affiche les erreurs tsc ou eslint directement dans le conv de la pr (encore faut-il que les droits du repo soit changé........)

@raphckrman
Copy link
Contributor Author

⚠️ Je viens une nouvelle fois de fix le Lint, merci de faire attention dans les push/PR car après c'est chiant pour le workflow

Tkt je l'ai amélioré le workflow de Typescript, voir => #449 Ça affiche les erreurs tsc ou eslint directement dans le conv de la pr (encore faut-il que les droits du repo soit changé........)

C'est cool ça mais la plupart des erreurs que j'ai fix ne viennent pas des PR mais des push direct sans verification du lint

@Kgeek33
Copy link
Contributor

Kgeek33 commented Dec 15, 2024

⚠️ Je viens une nouvelle fois de fix le Lint, merci de faire attention dans les push/PR car après c'est chiant pour le workflow

Tkt je l'ai amélioré le workflow de Typescript, voir => #449 Ça affiche les erreurs tsc ou eslint directement dans le conv de la pr (encore faut-il que les droits du repo soit changé........)

C'est cool ça mais la plupart des erreurs que j'ai fix ne viennent pas des PR mais des push direct sans verification du lint

Ah oui, ça c'est chiant 😭

@Kgeek33
Copy link
Contributor

Kgeek33 commented Dec 15, 2024

Comment tu fais eft pour que la review de tes prs soit si rapide ???
Perso, j'att une pr depuis 1 mois 😭😭 #346

@raphckrman
Copy link
Contributor Author

Comment tu fais eft pour que la review de tes prs soit si rapide ??? Perso, j'att une pr depuis 1 mois 😭😭 #346

ça dépend des fois mes PR font genre 5 lignes même pas

@Kgeek33
Copy link
Contributor

Kgeek33 commented Dec 15, 2024

Ok ouais je vois

@raphckrman
Copy link
Contributor Author

J'ai ajouté des images de preview pour faciliter les reviews

ecnivtwelve
ecnivtwelve previously approved these changes Dec 15, 2024
Copy link
Contributor

@ecnivtwelve ecnivtwelve left a comment

Choose a reason for hiding this comment

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

LGTM

tryon-dev
tryon-dev previously approved these changes Dec 16, 2024
@ecnivtwelve ecnivtwelve dismissed stale reviews from tryon-dev and themself via 5a66a93 December 17, 2024 18:16
@ecnivtwelve ecnivtwelve merged commit b6ab7e2 into PapillonApp:main Dec 17, 2024
1 check failed
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.

4 participants