[x] Cette pull request n'est pas un duplicata d'une autre
[x] Vous avez testé de build le projet avec vos modifications et ce build a réussi
[x] Vous respectez les conventions de codage et de nommage du projet
[x] Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
[x] Cette pull request est prête à être revue (review) et fusionnée (merge)
[x] Cette pull request doit être fusionnée dans la branche development (le cas contraire préciser laquelle)
[x] Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
[x] Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
[x] Les détails des changements ont été décrits ci-dessous
[x] Cette pull-request n'est pas une "breaking-change" (par exemple des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)
Explications des changements
Changements mineurs (2 lignes de code 😂):
Retour de la possibilité de modifier les valeurs stockées dans le LocalStorage, cette erreur incluse il y a 2 mois dans le commit de Vince utilisait des NativeItem qui n'avaient pas le style adapté, ce qui faisait que l'élément ne prenait pas toute la largeur, cela a fait que la View avec l'input était totalement masquée.
Il est désormais possible de pouvoir actualisé pour tous les jours les cours. La valeur de 150 choisi par Vince dans ce commit était trop élevée ce qui fait que l'élément n'était pas assez haut pour pouvoir permettre un défilement vertical qui est obligatoire pour pour pouvoir déclencher le système de refresh sur Android.
Ces réglages ont été testé sur un Z-flip 4 et un A32 5G, avec une densité modifiée sur le A32 permettant de simulé un écran large.
Changelogs proposés
Retour de l'édition du Local Storage.
Possibilité d'actualisé les cours pour tous les jours sur Android.
Note
Pour que votre pull-request soit fusionnée, vous devez obtenir l'approbation de au moins deux membres de l'équipe dont un coordinateur.
Soyez donc patient :)
Checklist d'avant pull request
development
(le cas contraire préciser laquelle)TODO
(aka des annotations pour du code manquant) dans vos modificationsExplications des changements
Changements mineurs (2 lignes de code 😂):
Retour de la possibilité de modifier les valeurs stockées dans le
LocalStorage
, cette erreur incluse il y a 2 mois dans le commit de Vince utilisait desNativeItem
qui n'avaient pas le style adapté, ce qui faisait que l'élément ne prenait pas toute la largeur, cela a fait que laView
avec l'input était totalement masquée.Il est désormais possible de pouvoir actualisé pour tous les jours les cours. La valeur de 150 choisi par Vince dans ce commit était trop élevée ce qui fait que l'élément n'était pas assez haut pour pouvoir permettre un défilement vertical qui est obligatoire pour pour pouvoir déclencher le système de refresh sur Android.
Ces réglages ont été testé sur un Z-flip 4 et un A32 5G, avec une densité modifiée sur le A32 permettant de simulé un écran large.
Changelogs proposés
Note
Pour que votre pull-request soit fusionnée, vous devez obtenir l'approbation de au moins deux membres de l'équipe dont un coordinateur. Soyez donc patient :)
Informations supplémentaires
https://github.com/PapillonApp/Papillon/assets/73659505/082af866-6e4b-4112-873b-93b4283cc170