Kgeek33 commented 3 weeks ago

[!NOTE]

Cette PR a été faite dans sa grande majorité par @Gabriel29306 (#294 ), mais closed suite à l'enchaînement de conflit, VU QUE PERSONNE FAISAIT DES REVIEWS !!!

Elle est complétée avec les derniers bugs trouvés

🚀 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].

[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 n'est pas un duplicata d'une autre
[x] Cette pull request est prête à être revue (review) et fusionnée (merge)
[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" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

Sur la PR #294

fix: Centrage des icones pour les cours, les devoirs, widget LastGrade fix: majuscules pour les noms de famille composés fix: nom de police: semiBold => semibold fix: icone de pièce jointe pour les communications fix: bouton soutenir Papillon dans les paramètres fix: chargement infini des actualités fix: utilisation des safe area insets pour les views ayant un style de modale refactor: Changement de l'animation pour les views ayant un style de modale refactor: réarrangement des onglets dans SettingsTabs refactor: configuration d'expo-font refactor: nouvelles lignes entre les salles de classes refactor: bouton de donation en Android only chore: Mise à jour des dépendances

Sur ma PR en sup.

Correction d'un bug qui n'affichait rien quand on sélectionne une date sur l'emploi du temps Optimisation du code Homogénéité entre Apple et Android sur setShowDatePicker(false); Comme à chaque maj de Papillon, les mises à jour de l'app sur Android pose problème, car la versionCode ne correspond pas à celle du Play Store En ce moment, @LeGeek01 a mis une version code de "7211", alors je la passe à "7300" Correction de l'affichage fonctionnant 1 fois sur 10 lorsqu'on clique sur un devoir BONUS: ajout d'animations sur d'autres pages pour éviter ce genre de problème Correction complète de la page d'accueil (marges, animations instables…) Passage RenderHTML -> HTMLView pour arrêter les 3 avertissements de Node (fonctionnalité JavaScript déprécié) Bump vers la version 7.4.0 + version code pour iOS modifiée pour homogénéité entre Apple et Android Correction couleur et émoji qui ne changeait pas lors de la modification d'une matière (permet ainsi l'affichage d'une alerte en cas d'aucun emoji) Généralisation de l'interface en passant de NativeText -> PapillonHeaderSelection sur Attendance.tsx (+ marginRight pour éviter que ça dépasse l'écran) Fix de la redirection vers la semaine prochaine alors qu'on est en plein milieu de la semaine (Devoirs) Fix des icônes qui ne s'affichait pas Fix de la moyenne générale qui peut être vide sur le widget Autres fix pour moins d'erreurs dans la console (exemple : FlatList)

Informations supplémentaires

Issues anciennes à closed

Sur la PR #294

Sur ma PR en sup.

Closed https://github.com/PapillonApp/Papillon/issues/310
Closes https://github.com/PapillonApp/Papillon/issues/312
Closes https://github.com/PapillonApp/Papillon/issues/232
Closes https://github.com/PapillonApp/Papillon/issues/315
Closes https://github.com/PapillonApp/Papillon/issues/311
Closes https://github.com/PapillonApp/Papillon/issues/318
Closes https://github.com/PapillonApp/Papillon/issues/331
Closes https://github.com/PapillonApp/Papillon/issues/336
Closes https://github.com/PapillonApp/Papillon/issues/338
Closes https://github.com/PapillonApp/Papillon/issues/332
Closes https://github.com/PapillonApp/Papillon/issues/350
Closes https://github.com/PapillonApp/Papillon/issues/358
Closes https://github.com/PapillonApp/Papillon/issues/367

Clmnnt commented 2 weeks ago

Je vais voir pour review maintenant :)

Clmnnt commented 2 weeks ago

Pour le calendrier ça passe tous les jours jusqu'au sélectionné ? Les animations sont lentes aussi, je sais pas si c'est normal (quand je dis lente, il reste 1sec après le clic pour afficher la page)

Kgeek33 commented 2 weeks ago

j'pense que c'est l'animation qui fait que ça balaye les jours jusqu'au jour sélectionné tu peux me montrer une vidéo pour que je vois ce que ça donne sur Apple stp ?

Clmnnt commented 2 weeks ago

Ça donne ça sur iOS 18: https://github.com/user-attachments/assets/ad680921-5dc8-4fdf-b913-db057a8da653

Kgeek33 commented 2 weeks ago

c toi qui fait les clics partout, rassure-moi ? 😅

Kgeek33 commented 2 weeks ago

@cleeemeeent j'ai trouvé le bug, je viens de le corriger (à cause d'une animation)

Clmnnt commented 2 weeks ago

c toi qui fait les clics partout, rassure-moi ? 😅

les clics partout ?

Kgeek33 commented 2 weeks ago

c toi qui fait les clics partout, rassure-moi ? 😅

les clics partout ?

quand j'ai vu ta vidéo, j'ai cru que tu t'amusais à cliquer sur n'importe quelle date alors que c'était juste un bug d'animation mdrr

Clmnnt commented 2 weeks ago

good les animations :)

Clmnnt commented 2 weeks ago

Petit problème ici :

Clmnnt commented 2 weeks ago

Le papillon ne s'affiche pas dans tous les onglets suivants

Kgeek33 commented 2 weeks ago

CA RECOMMENCE !? j'avais déjà rencontré ce bug mais jspas comment je l'avais résolu

Clmnnt commented 2 weeks ago

c'était sur les services cantines je crois

Kgeek33 commented 2 weeks ago

ben en fait, le bug, c'est qu'un module (jspas lequel) refuse d'afficher les images en local

Kgeek33 commented 2 weeks ago

go npm-check-updates, peut-être que je vais l'intégrer dans le package.json

Clmnnt commented 2 weeks ago

tu me tiens au courant comme ça je re-regarde

ecnivtwelve commented 2 weeks ago

ben en fait, le bug, c'est qu'un module (jspas lequel) refuse d'afficher les images en local

c'est @react-native/assets-registry, laisse le package json tel qu'il a été revert hier

Kgeek33 commented 2 weeks ago

@ecnivtwelve @cleeemeeent les images fonctionnent de nouveau à vous de review !

Clmnnt commented 2 weeks ago

je review après ! 😁

LeGeek01 commented 2 weeks ago

après plusieurs tests de mon côté, faute de review et parce que l'expérience android est médiocre sur la 7.3.0, j'ai publié la 7.5.0 bêta (pour correspondre avec test flight) sur cette pr sur le play store. par contre, où est passé la 7.4.0 ? il n'y a même pas de changelogs initiés

Kgeek33 commented 2 weeks ago

Oh cool merci bcp ! Sur la 7.4.0, à un moment, ils ont mis le Package.json en 7.4 mais un peu plus tard, jspas pq, c'est passé sur la 7.5

LeGeek01 commented 2 weeks ago

Bon tant pis en tout cas la version est en examen elle sera dispo dans le canal bêta après

Kgeek33 commented 2 weeks ago

OK super merci !

LeGeek01 commented 2 weeks ago

Bon l'examen dure longtemps je trouve

Kgeek33 commented 2 weeks ago

Oui c vrai, on avait l'habitude que ça dure 1/2 heure, mais pas autant quand même

Gabriel29306 commented 2 weeks ago

Bon l'examen dure longtemps je trouve

Je devine que tu ne peux pas faire une release sur le GitHub?

Kgeek33 commented 2 weeks ago

Non pas les droits Ça serait intéressant un github actions qui le fasse

Kgeek33 commented 2 weeks ago

@LeGeek01 @Gabriel29306 Ah mon père (ingénieur en informatique) vient de m'annoncer que l'analyse des apk/aab peut prendre désormais jusqu'à 2 jours !

Gabriel29306 commented 2 weeks ago

@LeGeek01 @Gabriel29306 Ah mon père (ingénieur en informatique) vient de m'annoncer que l'analyse des apk/aab peut prendre désormais jusqu'à 2 jours !

Et merde...

Kgeek33 commented 2 weeks ago

@LeGeek01 @Gabriel29306 Ah mon père (ingénieur en informatique) vient de m'annoncer que l'analyse des apk/aab peut prendre désormais jusqu'à 2 jours !

Et merde...

Google saoule

Gabriel29306 commented 2 weeks ago

C'est même plus dans certains cas:

Toutes les modifications apportées à une application doivent être traitées avant de pouvoir être publiées. Le traitement peut prendre quelques heures ou jusqu'à sept jours (ou plus dans des cas exceptionnels), car cela dépend du temps de révision auquel votre application est soumise.

https://support.google.com/googleplay/android-developer/answer/9859654

Kgeek33 commented 2 weeks ago

Wtf 7 jours !? Non la c'est trop là

Kgeek33 commented 2 weeks ago

@ecnivtwelve une review serait cool ! Ça fait déjà 11 jours...

LeGeek01 commented 2 weeks ago

La màj est dispo sur le play store

Kgeek33 commented 2 weeks ago

Je viens de voir ça !

LeGeek01 commented 2 weeks ago

On peut plus cocher les devoirs comme faits ?? J'avais pas ce bug moi

Kgeek33 commented 2 weeks ago

Oh non j'ai créé un bug 😭😭 Les devoirs peuvent plus être coché sur toutes les pages

Kgeek33 commented 2 weeks ago

Pourtant j'avais bien testé pour que ça soit uniquement sur la page d'accueil

Kgeek33 commented 2 weeks ago

Ah si OK j'ai compris pourquoi Je fais un fix de suite

Kgeek33 commented 2 weeks ago

OK c'est corrigé J'avais testé sur un appareil où les devoirs n'était pas sur la navbar, c'est pour ça, j'avais pas testé quand devoirs était dans la navbar

LeGeek01 commented 2 weeks ago

Tu préfères quoi encore une paire de claques ou une paire de claques ? T'as vu combien de temps la màj a pris pour passer ??? Putain x)

Kgeek33 commented 2 weeks ago

Oui bien oui, on va pas faire une nouvelle version juste pour un petit fix Je pourrai faire d'autres fixs s'il y a d'autres bugs (par exemple, en mode clair, on voit pas le nom du compte)

Gabriel29306 commented 2 weeks ago

Étant sur la version GitHub j'ai pas de problèmes

Kgeek33 commented 2 weeks ago

avec les devoirs ? et pour le bug connu, faut être en mode clair et aller sur la page d'accueil, faire défiler. On voit extrêmement mal le nom du compte

LeGeek01 commented 2 weeks ago

Étant sur la version GitHub j'ai pas de problèmes

Sur cette pr ? @Gabriel29306

Kgeek33 commented 2 weeks ago

après j'ai fait un commit ce matin, c'est peut-être pour ça fonctionne

Gabriel29306 commented 2 weeks ago

Étant sur la version GitHub j'ai pas de problèmes

Sur cette pr ? @Gabriel29306

Ah non, la release officielle

ecnivtwelve commented 1 week ago

Je précise que la review de la PR est en cours de mon côté (et de celui de @Vexcited je crois bien), c'est juste encore une fois qu'il y a de nombreuses choses à vérifier comme la PR est majeure.

LeGeek01 commented 1 week ago

Je précise que la review de la PR est en cours de mon côté (et de celui de @Vexcited je crois bien), c'est juste encore une fois qu'il y a de nombreuses choses à vérifier comme la PR est majeure.

au lieu de nous laisser dans le vent, c'est une info à donner dès que vous commencez ça

Kgeek33 commented 1 week ago

@ecnivtwelve @Gabriel29306 @LeGeek01 Alors là, je viens d'apprendre la meilleure On peut plus tester Papillon, faut monter Expo en version 52. Jspas si ça nécessite les nouvelles versions majeures de React /React-native, où il faudra reprendre la majorité du code

PapillonApp / Papillon

fixs: Tous les bugs existants #325

🚀 Nouvelle Pull Request

Informations importantes

Checklist d'avant pull request

Changelogs proposés

Sur la PR #294

Sur ma PR en sup.

Informations supplémentaires

Issues anciennes à closed

Sur la PR #294

Sur ma PR en sup.