Mise à jour vers Expo 52

[raphckrman]

🚀 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

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

[Kgeek33]

c'est mieux que tout à l'heure mdrr
avant que t'appliques mon autre commentaire, mets à jour tous les packages vers la dernière version en exécutant ncu -i (version interactive) ou ncu (affiche les packages avec une mise à jour disponible)

Comme se sont des versions majeures, je te conseille de mettre à joour les packages 1 par 1, car je crois qu'un module nécessite une modification de code, jsplus lequel

[Kgeek33]

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

[Kgeek33]

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

[raphckrman]

c'est good

[Kgeek33]

Fais cette modif, mais sinon LGTM :)

[Kgeek33]

Sur cette ligne là, tu dois faire la modification de la version de ESLint : https://github.com/raphckrman/PapillonV7/blob/ee8f6d831e44c21182364f42a04b440cefdc64ed/package-lock.json#L8893

"eslint": "^3.17.0 || ^4 || ^5 || ^6 || ^7 || ^8" => "eslint": "^3.17.0 || ^4 || ^5 || ^6 || ^7 || ^8 || ^9"

[Kgeek33]

LGTM 👍

[raphckrman]

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]

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é

[raphckrman]

On peut la passer en Draft en si tu veux histoire de faire ça bien ? Ou attendre l'avis d'un mainteneur qu'on sache directement si il vaut mieux close pour qu'ils le fassent dans une V8 comme avait suggéré Tryon ? Comme tu veux

À la limite ouais on passe la pr en draft mais pendant ce temps, faut relancer les mainteneurs notamment pask on est vrmt dans le flou 🤣 En plus, React Native 0.77 sort bientôt, j'vous laisse imaginer tous les problèmes qu'on va avoir, déjà qu'on galère avec la 0.76 😂😂

Yes c'est pour ça a voir avec les mainteneurs, je switch en draft

[Kgeek33]

Ok tu me diras quand tu seras prêt à rewrok ça ! Perso, je suis en train de faire de coder une belle nouvelle version de mon site (j'suis le seul à coder min site donc pas de deadline, si p'tetre avant la rentrée je me fixe)

Mdrrr mais je m'impose des nouvelles fonctionnalités sur mon site qui prennent énormément de temps à être développé 😂

[Kgeek33]

Yes c'est pour ça a voir avec les mainteneurs, je switch en draft

Okay parfait ben d'ici à ce qu'on ait des réponses, mais j'vais faire une pause sur Papillon (juste mettre à jour mes branches) et me concacrer à mon site (et à la rentrée bien évidemment 😒)
Et toi @raphckrman ?

[raphckrman]

Yes c'est pour ça a voir avec les mainteneurs, je switch en draft

Okay parfait ben d'ici à ce qu'on ait des réponses, mais j'vais faire une pause sur Papillon (juste mettre à jour mes branches) et me concacrer à mon site (et à la rentrée bien évidemment 😒)
Et toi @raphckrman ?

J'ai une mise à jour prévu pour Papillon et après c'est full Swift pour le Swift Student Challenge il reste plus beaucoup de temps 🤓

[Kgeek33]

Yes c'est pour ça a voir avec les mainteneurs, je switch en draft

Okay parfait ben d'ici à ce qu'on ait des réponses, mais j'vais faire une pause sur Papillon (juste mettre à jour mes branches) et me concacrer à mon site (et à la rentrée bien évidemment 😒)
Et toi @raphckrman ?

J'ai une mise à jour prévu pour Papillon et après c'est full Swift pour le Swift Student Challenge il reste plus beaucoup de temps 🤓

Oh c'est super ça, bon courage d'avance !
Bon, c'est pas tout ça, mais on a bien mérité notre nuit de sommeil mdrrr 😂😂

[Kgeek33]

hey @raphckrman j'ai des infos pour toi (plutôt bonnes) sur cette pr

[raphckrman]

hey @raphckrman j'ai des infos pour toi (plutôt bonnes) sur cette pr

Tu as trouvé une solution ?

[Kgeek33]

hey @raphckrman j'ai des infos pour toi (plutôt bonnes) sur cette pr

Tu as trouvé une solution ?

alors depuis tout à l'heure je suis en train de réécrire le router, je peux faire un commit pour te montrer stv (j'ai crée un dossier router-v2 en conservant le dossier de base pour faire une réécriture progressive)
et aussi, j'ai trouvé les sources de plantage lorsqu'on testait sur Expo 52 : c'est le module react-native-gesture-handler qui pose problème. mais quand on enlève la balise <GestureHandlerRootView>, il y a l'erreur suivante et je cherche quel fichier provoque l'erreur :

ERROR Warning: Error: NativeViewGestureHandler must be used as a descendant of GestureHandlerRootView. Otherwise the gestures will not be recognized. See https://docs.swmansion.com/react-native-gesture-handler/docs/installation for more details.

[raphckrman]

incroyable tu as déjà push le début du nouveau router ?

[raphckrman]

hey @raphckrman j'ai des infos pour toi (plutôt bonnes) sur cette pr

Tu as trouvé une solution ?

alors depuis tout à l'heure je suis en train de réécrire le router, je peux faire un commit pour te montrer stv (j'ai crée un dossier router-v2 en conservant le dossier de base pour faire une réécriture progressive) et aussi, j'ai trouvé les sources de plantage lorsqu'on testait sur Expo 52 : c'est le module react-native-gesture-handler qui pose problème. mais quand on enlève la balise <GestureHandlerRootView>, il y a l'erreur suivante et je cherche quel fichier provoque l'erreur :

ERROR Warning: Error: NativeViewGestureHandler must be used as a descendant of GestureHandlerRootView. Otherwise the gestures will not be recognized. See https://docs.swmansion.com/react-native-gesture-handler/docs/installation for more details.

L'erreur ne pourrait pas venir de TouchableOpacity?

[Kgeek33]

incroyable tu as déjà push le début du nouveau router ?

je vais le faire là, mais je préviens l'affichage est un peu moche 😅

[Kgeek33]

L'erreur ne pourrait pas venir de TouchableOpacity?

jspas mais en tout cas, tout ce que j'ai dit est faux, je viens de tester mdrrr

[Kgeek33]

et voilà, il y a des erreurs tsc, ça sera corrigé plus tard

[tom-theret]

@Kgeek33 comment on peut faire pour te contacter en PV ?

[Kgeek33]

@Kgeek33 comment on peut faire pour te contacter en PV ?

ah jspas, j'y ai jamais réfléchi 😅

[tom-theret]

@Kgeek33 comment on peut faire pour te contacter en PV ?

ah jspas, j'y ai jamais réfléchi 😅

Bah donne moi un moyen 😅

[Kgeek33]

Bah donne moi un moyen 😅

mon num, ça te va ? (et tu pourras me contacter sur whatsapp, plus pratique)

[tom-theret]

@Kgeek33

[Kgeek33]

ok j'tenvoie un message (jsuis en train de faire un fix sur les bugs html en même temps)

[Kgeek33]

okay @raphckrman @tom-theret j'ai des informations pour vous !

alors j'pense pas poursuivre le nouveau router car j'ai fait des optimisations sur le fichier App.tsx qui améliore les perfs sur Expo 52 (avec test y compris sur iOS) => voir commit 5cc2e76

• Pour iOS, j'ai trouvé la source du problème au niveau du rendu graphique et attention, vous allez être choqué… c'est Reanimated qui pose (encore) problème ! Donc demain, je désactive tous les effets d'animation effectués par Reanimated (ça sera moche mais stable)
• Pour Android (j'ai pas testé pour iOS), cependant, quand une erreur est rencontré, Expo ne répond plus et pour l'instant je n'ai pas trouvé de pistes…

faites des tests (tout le monde svp) et dites ce qu'il faut corriger, mis à part les problèmes graphiques sur iOS :)

[tom-theret]

Sauf qu'on a besoin des animations, c'est à ça qu'on reconnaît le tacts de Papillon

[Kgeek33]

Sauf qu'on a besoin des animations, c'est à ça qu'on reconnaît le tacts de Papillon

Mouais c'est ça le problème, il n'y aurait pas une alternative à Reanimated tout en faisant des animations ?