PapillonApp / Papillon

Une alternative open source aux applis de vie scolaire. L'allié de tous les étudiants. Le futur de l'éducation numérique libre et ouverte.
https://papillon.bzh
GNU General Public License v3.0
112 stars 42 forks source link

Mise à jour vers Expo 52 #370

Open raphckrman opened 1 week ago

raphckrman commented 1 week ago

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

Changelogs proposés

Passez sur Expo 52 pour des meilleurs perfs et continuer à tester via Expo Go.

Kgeek33 commented 1 week ago

et tu peux mettre le 1er commentaire en résolu

Kgeek33 commented 1 week ago

quand t'auras fait ça, relance-moi 😉

raphckrman commented 1 week ago

c'est good

raphckrman commented 1 week ago

je viens de remarquer qu'avec l'update des packages de kgeek ça fonctionne plus je vais regarder quel module pose problème demain

Kgeek33 commented 1 week ago

Perso, l'application fonctionne mais des pages ne fonctionnent plus :

Je vais refaire une review pour corriger les problèmes de sécurité sur npm et (si j'ai le temps) corriger les bugs que j'ai listé

Kgeek33 commented 1 week ago

Si tu rencontres un blocage d'Expo lors des tests, c'est pas Papillon qui déconne (enfin j'espère) Elle plante, faut la redémarrer pour qu'elle fonctionne

raphckrman commented 6 days ago

Perso, l'application fonctionne mais des pages ne fonctionnent plus :

  • Onglets et Navigation => extrêmement lent
  • Matières => affichage des options lorsqu'on clique sur une matière très bizarre
  • Sélection d'une matière => la couleur sélectionnée ne fonctionne pas
  • Toutes les autres page avec un défilement => ne scoll pas jusqu'au bout, donc des infos sont coupées

Je vais refaire une review pour corriger les problèmes de sécurité sur npm et (si j'ai le temps) corriger les bugs que j'ai listé

Je vais regarder pour fixer tout ça, à noter que Expo 52 est très récent et qu'il y a actuellement pas mal d'issues concernant Expo 52, j'avais aussi une application très lente et avec des gros glitchs sur Expo Go, depuis la réinstallation d'Expo Go je n'ai plus cette lenteur et les glitchs.

raphckrman commented 6 days ago

J'ai un bug sur Expo Go que j'ai pas sur le simulateur, tu as aussi des bugs visuels genre des noms de matières dans le vide sur certaines pages etc?

Kgeek33 commented 6 days ago

Perso, l'application fonctionne mais des pages ne fonctionnent plus :

  • Onglets et Navigation => extrêmement lent
  • Matières => affichage des options lorsqu'on clique sur une matière très bizarre
  • Sélection d'une matière => la couleur sélectionnée ne fonctionne pas
  • Toutes les autres page avec un défilement => ne scoll pas jusqu'au bout, donc des infos sont coupées

Je vais refaire une review pour corriger les problèmes de sécurité sur npm et (si j'ai le temps) corriger les bugs que j'ai listé

Je vais regarder pour fixer tout ça, à noter que Expo 52 est très récent et qu'il y a actuellement pas mal d'issues concernant Expo 52, j'avais aussi une application très lente et avec des gros glitchs sur Expo Go, depuis la réinstallation d'Expo Go je n'ai plus cette lenteur et les glitchs.

Oui c'est pas surprenant qu'il y ait bcp d'issues, c'est dommage que ça soit brutale la migration, sans laisser genre 1 mois pour migration sdk 51 => 52 Pour tester en ce moment, j'ai fait un downgrade de l'app prenant en charge sdk 51. Sur l'autre appareil, j'ai fait la même et je l'ai mis à jour vers la dernière, c'est peut-être pour ça les plantages

Kgeek33 commented 6 days ago

J'ai un bug sur Expo Go que j'ai pas sur le simulateur, tu as aussi des bugs visuels genre des noms de matières dans le vide sur certaines pages etc?

Oui, comme des bugs extrêmement aléatoire de l'affichage de certaines pages, mais ça je pense que c'est du à React Native et non expo

raphckrman commented 6 days ago

Je viens de push les fixs que tu m'as dis sur tu en remarque d'autre n'hésite pas à me le dire, je vais essayer de regarder aussi mais pour l'instant c'est plutôt instable à voir avec les MaJs de Expo et React Native

Kgeek33 commented 6 days ago

Oui j'hésiterai pas à te signaler Mais il va encore falloir changer des choses, car :

Bref, à revoir certains points...

raphckrman commented 6 days ago

Je regarderais ça quand j'aurais un peu de temps, si tu veux je peux t'ajouter dans mon fork si tu veux fix des bugs dessus pour les rajouter ici

Kgeek33 commented 6 days ago

Ah oui, ca serait cool 😁 En plus, demain aprem, j'ai pas cours donc je pourrai corriger des choses

Kgeek33 commented 6 days ago

@Vexcited @ecnivtwelve Avant que j'essaye de corriger des choses, vous pouvez review ? Pas bcp de commits mais bcp de lignes modifiées 😅

raphckrman commented 6 days ago

Je t'ai invité sur le fork, y'a pas énormément de lignes la plupart c'est juste les package.json, package-lock.json

Kgeek33 commented 6 days ago

Oui comme tous les packages ont été mis à jour vers la dernière version

Kgeek33 commented 6 days ago

OK ben j'essaye de corriger des bugs demain aprem, j'pense qu'il va y avoir un peu de travail 😂

Kgeek33 commented 6 days ago

bon j'ai corrigé plusieurs bugs par contre, il en reste énormément je te recommande @raphckrman de mettre cette pr en draft, car c'est extrêmement compliqué de tester avec Expo 52 et React-Native (l'appli plante extrêmement facilement)

Pour le moment, pour tester sur Expo 51, il faut télécharger la version précédente (https://apkpure.com/expo-go/host.exp.exponent/downloading/2.31.1) et désinstaller celle actuelle pour l'apk

raphckrman commented 5 days ago

Comment l'appli plante? J'ai aucun crash de mon côté, après je suis sur iOS je sais pas si ça change quelque chose, je la passe en draft

Kgeek33 commented 5 days ago

En gros, lors des tests, expo va fonctionner normalement et quand tu redémarres l'appli (expo donc), Papillon va figer et l'app ça planter complètement, sans raison Problème du côté d'expo 52 du coup

Kgeek33 commented 1 day ago

@raphckrman tu peux réouvrir la pr mais 3 points à noter : (je teste sur Android, pas encore testé sur iOS)

[!WARNING]

Lorsque vous rencontrez une erreur, que vous corrigez l'erreur, Expo va vous redirigez vers la page du choix des comptes. Redémarrer l'app, sinon Expo va planter

Oui, c'est compliqué de tester avec Expo 52