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

Better attendance date #631

Merged
merged 3 commits into from
Feb 8, 2025
Merged

Conversation

JyhuKo
Copy link
Contributor

@JyhuKo JyhuKo commented Jan 10, 2025

🚀 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

ajout de la de fin d une absence

Informations supplémentaires

image

Copy link
Contributor

@Gabriel29306 Gabriel29306 left a comment

Choose a reason for hiding this comment

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

LGTM

@ecnivtwelve
Copy link
Contributor

un peu chargé visuellement non ? @toi-et-moi

@JyhuKo
Copy link
Contributor Author

JyhuKo commented Jan 20, 2025

un peu chargé visuellement non ? @toi-et-moi

c est mieuxc comme ca ?
image
si c est le meme jour ca ca met juste une date et sinon ca met sur deux lignes

@ecnivtwelve
Copy link
Contributor

l'affichage en DD/MM/YY réduit est pas très clair

@JyhuKo
Copy link
Contributor Author

JyhuKo commented Jan 20, 2025

dcp plutot mettre?
Du lundi 23 janvier 2024 à 17:00
Au mardi 24 janvier 2024 à 17:00
et l 'année seulement si l année de l abcense n est pas l année actuelle

@JyhuKo JyhuKo closed this Jan 20, 2025
@JyhuKo JyhuKo force-pushed the BetterAttendanceDate branch from 90153bf to 51e85b1 Compare January 20, 2025 16:26
j ai rienc chnagé c etait juste pour update la branch
@JyhuKo JyhuKo reopened this Jan 20, 2025
@JyhuKo
Copy link
Contributor Author

JyhuKo commented Jan 20, 2025

comment vous faites vos truc merge branch main to ---

@NathanBnm
Copy link
Contributor

comment vous faites vos truc merge branch main to ---

@JyhuKo tu ne peux pas approuver et fusionner tes propres modifications il faut que ce soit validé et fait par les mainteneurs du projet ;)

@JyhuKo
Copy link
Contributor Author

JyhuKo commented Jan 20, 2025

comment vous faites vos truc merge branch main to ---

@JyhuKo tu ne peux pas approuver et fusionner tes propres modifications il faut que ce soit validé et fait par les mainteneurs du projet ;)

oui je sais mais je parle de ca
image

@NathanBnm
Copy link
Contributor

comment vous faites vos truc merge branch main to ---

@JyhuKo tu ne peux pas approuver et fusionner tes propres modifications il faut que ce soit validé et fait par les mainteneurs du projet ;)

oui je sais mais je parle de ca
image

Il faut que tu mettes à jour ton fork avec le dépôt distant (tu peux le faire directement depuis la page de ton fork en haut à droite)

Et pour mettre à jour ta branche vis à vis de la branche main (mettre les modifications de main dans ta branche) tu peux faire git merge main quand tu est sur ta branche

@JyhuKo
Copy link
Contributor Author

JyhuKo commented Jan 20, 2025

comment vous faites vos truc merge branch main to ---

@JyhuKo tu ne peux pas approuver et fusionner tes propres modifications il faut que ce soit validé et fait par les mainteneurs du projet ;)

oui je sais mais je parle de ca
image

Il faut que tu mettes à jour ton fork avec le dépôt distant (tu peux le faire directement depuis la page de ton fork en haut à droite)

Et pour mettre à jour ta branche vis à vis de la branche main (mettre les modifications de main dans ta branche) tu peux faire git merge main quand tu est sur ta branche

okok merci

@tryon-dev tryon-dev merged commit 415d8b8 into PapillonApp:main Feb 8, 2025
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.

6 participants