Mise à jour vers Expo 52 #370

raphckrman · 2024-11-16T18:33:39Z

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

… module

package.json

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

package.json

Kgeek33 · 2024-11-16T20:21:09Z

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

Co-authored-by: 𝕂𝕪𝕝𝕚𝕒𝕟 <[email protected]>

Kgeek33 · 2024-11-16T20:37:36Z

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

raphckrman · 2024-11-16T20:41:31Z

c'est good

Kgeek33

Fais cette modif, mais sinon LGTM :)

package-lock.json

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"

package.json

Kgeek33

LGTM 👍

src/components/Global/PressableScale.tsx

raphckrman · 2024-11-17T22:20:29Z

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 · 2024-11-18T18:34:16Z

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é

Clmnnt · 2024-12-31T21:04:19Z

alors des news ?

Kgeek33 · 2024-12-31T21:06:42Z

alors des news ?

Côté Android, j'ai fait des tests build tout à l'heure et c'est rempli de plantage
Même Expo est un peu plus stable (mais rempli de plantages également)

Clmnnt · 2024-12-31T21:07:05Z

c'est quoi qui fait bug ?

Clmnnt · 2024-12-31T21:07:53Z

(de mon coté j'essaye de lancer avec expo go sur iOS)

Kgeek33 · 2024-12-31T21:11:18Z

c'est quoi qui fait bug ?

Je pense que c'est react-native-reanimated (c'est sûr pour les problèmes graphiques sur ios) mais ça peut être aussi le router qui est incompatible et qu'il faut reprendre à 0 (pour les plantages)

Clmnnt · 2024-12-31T21:13:23Z

mais Papillon sans animations c'est pas papillon hein :bruh:

Clmnnt · 2024-12-31T21:15:44Z

ah ouais en effet c plein de bugs

Kgeek33 · 2024-12-31T21:17:18Z

mais Papillon sans animations c'est pas papillon hein :bruh:

Oui c'est ça le problème, ça sera extrêmement moche à utiliser Papillon
Faudrait trouver soit une alternative à ce module, soit (ou les 2) complètement réécrire le router (car j'avais constaté que le module react-native-gesture-handler provoquait des plantages lorsque j'avais commencé à réécrire le router)

Kgeek33 · 2024-12-31T21:18:27Z

ah ouais en effet c plein de bugs

Mais le problème c'est qu'il a plein de bugs sur Expo qui ne sont pas présents sur build et inversement donc on est bloqué...

Clmnnt · 2024-12-31T21:18:39Z

ouais mais réécrire un router c pas super long ?

Kgeek33 · 2024-12-31T21:24:00Z

ouais mais réécrire un router c pas super long ?

Si, très long. Après, j'ai commencé à le réécrire la semaine dernière donc je peux poursuivre

Kgeek33 · 2025-01-01T11:30:32Z

Yo tlm, déjà bonne année !
voici un fil Reddit que j'ai trouvé parlant de la mise à niveau vers Expo 52 : https://www.reddit.com/r/expo/comments/1gqblmj/expo_sdk_52/?tl=fr

Cela confirme bien nos pistes mais je m'attendais pas que SafeAreaView pose problème, je m'en suis pas rendu compte lors de mes tests

Gabriel29306 · 2025-01-01T12:51:58Z

Je sens que ça va être (encore plus) long...

Kgeek33 · 2025-01-01T21:03:04Z

ok alors j'ai fait un logcat sur Android pour savoir qu'est-ce qui provoquait les plantages (testé via build). Et voici l'erreur retourné lors d'un plantage (juste lors de l'ouverture de l'app...)

reanimated::LayoutAnimationsManager::startLayoutAnimation

Clmnnt · 2025-01-04T20:21:53Z

aie coup dur pour papillon qui aura jamais expo 52 :zjesus:

JyhuKo · 2025-01-04T20:23:02Z

aie coup dur pour papillon qui aura jamais expo 52 :zjesus:

pourquoi tu met des emojis discord ???

Clmnnt · 2025-01-04T20:24:04Z

je voulais voir si les emojis s'affichaient dans le webhook discord (spoiler : non)

Kgeek33 · 2025-01-04T20:27:16Z

aie coup dur pour papillon qui aura jamais expo 52 :zjesus:

Bon de toute façon, j'en ai discuté avec @raphckrman mais cette branche est cassé (des choses supprimées qui ont été push sur le main) lors des merges à cause des conflits en continu, faut refaire une branche et reprendre les modifs

JyhuKo · 2025-01-04T20:27:27Z

je voulais voir si les emojis s'affichaient dans le webhook discord (spoiler : non)

dommage :( att jessaye un truc <:zjesus:1147231011881615461>

JyhuKo · 2025-01-04T20:27:34Z

ezzzzzzzzzzz

Kgeek33 · 2025-01-04T20:27:43Z

je voulais voir si les emojis s'affichaient dans le webhook discord (spoiler : non)

dommage :( att jessaye un truc <:zjesus:1147231011881615461>

mais mdrrrr

JyhuKo · 2025-01-04T20:27:46Z

jsuis chaud

Clmnnt · 2025-01-04T20:27:57Z

ouais y’a des conflicts

Kgeek33 · 2025-01-04T20:29:22Z

ouais y’a des conflicts

@raphckrman on reprend à zéro toutes les modifs sur une nouvelle pr ? risqué de faire un revert depuis un commit

JyhuKo · 2025-01-04T20:34:04Z

aie coup dur pour papillon qui aura jamais expo 52 :zjesus:

Bon de toute façon, j'en ai discuté avec @raphckrman mais cette branche est cassé (des choses supprimées qui ont été push sur le main) lors des merges à cause des conflits en continu, faut refaire une branche et reprendre les modifs

c est pour ca que le graphique des notes s'affiche plus ?

Kgeek33 · 2025-01-04T21:10:02Z

aie coup dur pour papillon qui aura jamais expo 52 :zjesus:

Bon de toute façon, j'en ai discuté avec @raphckrman mais cette branche est cassé (des choses supprimées qui ont été push sur le main) lors des merges à cause des conflits en continu, faut refaire une branche et reprendre les modifs

c est pour ca que le graphique des notes s'affiche plus ?

Oui c'est désactivé sur Expo : sur ios pas de problème (lors de mes derniers tests) et sur Android ça provoque une erreur qui entraîne un plantage complet d'expo

JyhuKo · 2025-01-05T18:06:53Z

du coup il ba ce passer quoi sur cette pr ???
<:moipls:1216817535153078334>

Kgeek33 · 2025-01-05T18:17:26Z

du coup il ba ce passer quoi sur cette pr ??? <:moipls:1216817535153078334>

je vais en discuter avec Raph, pour le moment va falloir se contenter d'Expo 51 (donc des simulateurs :zjesus: )
Mais si vous utilisez Android, vous pouvez tjrs faire un downgrade d'Expo Go pour pouvoir tester Papillon :)

tryon-dev · 2025-01-05T19:34:54Z

Hello, malheureusement a moins de trouver la solution miracle Papillon restera sur expo 51

raphckrman added 5 commits November 15, 2024 00:00

chore: update to Expo 52

ead3071

fix: lint + AnimatedScrollView

75ab10a

fix(pressablescale): create a fixed component and remove the outdated…

97ac0a3

… module

chore: add ncu

8eab36d

fix(lint)

e5de353

raphckrman requested review from godetremy, tryon-dev and ecnivtwelve as code owners November 16, 2024 18:33

Kgeek33 reviewed Nov 16, 2024

View reviewed changes

package.json Outdated Show resolved Hide resolved

chore: update package.json

7164454

Kgeek33 reviewed Nov 16, 2024

View reviewed changes

package.json Outdated Show resolved Hide resolved

Update package.json

8578d06

Co-authored-by: 𝕂𝕪𝕝𝕚𝕒𝕟 <[email protected]>

chore: update packages

64d2e96

raphckrman added 4 commits November 16, 2024 21:49

fix(types): types error due to update of openid-client

237871c

chore: update for iOS 15.1

869e740

fix(navigation): add descriptors

04be25b

Merge branch 'main' into chore/expo52

3e8ece5

Kgeek33 reviewed Nov 17, 2024

View reviewed changes

package-lock.json Outdated Show resolved Hide resolved

raphckrman added 3 commits November 17, 2024 15:16

chore: update packages

949c265

chore: add react-navigation/native

6526a29

chore: add react-navigation/native

ee8f6d8

Kgeek33 reviewed Nov 17, 2024

View reviewed changes

package.json Outdated Show resolved Hide resolved

package.json Outdated Show resolved Hide resolved

chore: update packages

f256564

Kgeek33 approved these changes Nov 17, 2024

View reviewed changes

ecnivtwelve reviewed Nov 17, 2024

View reviewed changes

src/components/Global/PressableScale.tsx Show resolved Hide resolved

bump to latest version

efcbce5

Mise à jour vers Expo 52 #370

Are you sure you want to change the base?

Mise à jour vers Expo 52 #370

Conversation

raphckrman commented Nov 16, 2024

🚀 Nouvelle Pull Request

Informations importantes

Checklist d'avant pull request

Changelogs proposés

Kgeek33 left a comment

Choose a reason for hiding this comment

Kgeek33 commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

raphckrman commented Nov 16, 2024

Kgeek33 left a comment

Choose a reason for hiding this comment

Kgeek33 left a comment

Choose a reason for hiding this comment

Kgeek33 left a comment

Choose a reason for hiding this comment

raphckrman commented Nov 17, 2024

Kgeek33 commented Nov 18, 2024

Clmnnt commented Dec 31, 2024

Kgeek33 commented Dec 31, 2024

Clmnnt commented Dec 31, 2024

Clmnnt commented Dec 31, 2024

Kgeek33 commented Dec 31, 2024

Clmnnt commented Dec 31, 2024

Clmnnt commented Dec 31, 2024

Kgeek33 commented Dec 31, 2024

Kgeek33 commented Dec 31, 2024

Clmnnt commented Dec 31, 2024

Kgeek33 commented Dec 31, 2024

Kgeek33 commented Jan 1, 2025 • edited Loading

Gabriel29306 commented Jan 1, 2025

Kgeek33 commented Jan 1, 2025

Clmnnt commented Jan 4, 2025

JyhuKo commented Jan 4, 2025

Clmnnt commented Jan 4, 2025

Kgeek33 commented Jan 4, 2025

JyhuKo commented Jan 4, 2025

JyhuKo commented Jan 4, 2025

Kgeek33 commented Jan 4, 2025

JyhuKo commented Jan 4, 2025

Clmnnt commented Jan 4, 2025

Kgeek33 commented Jan 4, 2025

JyhuKo commented Jan 4, 2025

Kgeek33 commented Jan 4, 2025

JyhuKo commented Jan 5, 2025

Kgeek33 commented Jan 5, 2025

tryon-dev commented Jan 5, 2025

Kgeek33 commented Jan 1, 2025 •

edited

Loading