Better Home Lessons #535
Better Home Lessons #535
Conversation
biensur |
|
lets gooo |
Co-authored-by: Raphaël <[email protected]>
Co-authored-by: Raphaël <[email protected]>
| @@ -120,7 +123,7 @@ export const Page = ({ day, date, current, paddingTop, refreshAction, loading, w | |||
| )} | |||
|
|
|||
| {day.length === 1 && current && !loading && (day[0].type === "vacation" ? <MissingItem | |||
| emoji="🏝️" | |||
| emoji={emoji} | |||
| title="C'est les vacances !" | |||
| description="Profitez de vos vacances, à bientôt." | |||
There was a problem hiding this comment.
Ce serait intéressant d'indiquer la durée des vacances:
"Retour en classe le [dernier jour sans cours +1], d'ici là profitez de vos vacances, à bientôt."
Mais sur la page d'accueil juste, dans le petit popup de l'agenda
Et par contre pourquoi pas avoir plus d'infos dans l'agenda du style "Vacances de [nom vacances] du [date début complète (ex: 21 décembre)] au [date fin complète (ex: 6 janvier)]"
There was a problem hiding this comment.
juste faire [Dernier jour + 1] ca marchera vraiment ? car le dernier jour c'est pas le vendredi ? et les cours recommence que le lundi, or cela indiquerai le samedi
There was a problem hiding this comment.
Oui mais en comptant aussi les jours sans cours du coup
En gros il regarde sur les prochains jours après le dernier jour des vacances, il compte le nombre de jours sans cours (ex: 1 ou 2) en partant du premier jour qu'il regarde, et il ajoute le nombre au nombre de jours de vacances, par contre si il ne détecte pas de jours sans cours en partant du début, il ajoute 0 car c'est le nombre de jours consécutifs sans cours à partir du premier jour
Du style les vacances s'arrêtent vendredi, le programme regarde sur les 7 prochains jours, il détecte des jours consécutifs sans cours au début de son échantillon de jours (Donc jour 1 et jour 2 il trouve des jours sans cours) -> il les ajoute au nombre de jours des vacances
There was a problem hiding this comment.
ça consommerait trop de ressources réseau un peu inutiles non ?
There was a problem hiding this comment.
Effectivement, même si c'est une bonne idée, cela consommerait beaucoup de ressources et après, ça va prendre trop de temps pour retourner la durée
Co-authored-by: Vinicel <[email protected]>
Co-authored-by: Vinicel <[email protected]>
🚀 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].
TODO(aka des annotations pour du code manquant) dans vos modificationsChangelogs proposés
Ajout des vacances et du weekend dans l'accueil comme dans l'onglet cours et ajout d'emojis différents pour chaque vacances
Informations supplémentaires
avant (avec ecoledirecte):
apres:
Issues en lien
close #525