Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mise à jour vers Expo 52 #370

Draft
wants to merge 61 commits into
base: main
Choose a base branch
from

Conversation

raphckrman
Copy link
Contributor

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

package.json Outdated Show resolved Hide resolved
Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 Outdated Show resolved Hide resolved
@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 16, 2024

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

Co-authored-by: 𝕂𝕪𝕝𝕚𝕒𝕟 <[email protected]>
@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 16, 2024

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

@raphckrman
Copy link
Contributor Author

c'est good

Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fais cette modif, mais sinon LGTM :)

package-lock.json Outdated Show resolved Hide resolved
Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@raphckrman
Copy link
Contributor Author

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
Copy link
Contributor

Kgeek33 commented Nov 18, 2024

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é

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 26, 2024

@ecnivtwelve j'ai vu que t'y travailles également
Tu veux pas intégrer tes modifs dans la pr pour plus d'efficacité (ou inversement)?

@ecnivtwelve
Copy link
Contributor

@ecnivtwelve j'ai vu que t'y travailles également Tu veux pas intégrer tes modifs dans la pr pour plus d'efficacité (ou inversement)?

yes!

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 27, 2024

j'ai merged ta branche directement dans cette pr, plusieurs fixs (notamment les conflits) seront résolus plus tard

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 27, 2024

@raphckrman @ecnivtwelve @Vexcited @tryon-dev
À vous de review (oui, il y a un peu de travail 😅) pour la compatibilité avec Expo 52 et la nouvelle architecture de React Native. Un bug gênant reste tout de même présent :
Dans la page Onglets et navigation dans les paramètres de Papillon, le déplacement des matières ne va plus fonctionner (sûrement un problème de Reanimated, qui recommande d'utiliser useSharedValue)

@Clmnnt
Copy link
Contributor

Clmnnt commented Nov 30, 2024

J'ai toujours juste l'icone sur l'écran de démarrage, normal ?

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 30, 2024

C'est une "nouveauté" d'expo (que je n'aime pas du tout, rendu extrêmement moche)

Copy link
Contributor

@Clmnnt Clmnnt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Clmnnt
Copy link
Contributor

Clmnnt commented Nov 30, 2024

trim.E3EEE0BF-5E44-4A45-9A9B-71F16F88F03E.MOV

Ici aussi, il y a un problème

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 30, 2024

Il y a un bug ici https://github.com/user-attachments/assets/c8f391be-a9fb-4847-88ea-bea7396ba513

ah, il me tarde de commencer ma pr #378 🤣🤣

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 30, 2024

trim.E3EEE0BF-5E44-4A45-9A9B-71F16F88F03E.MOV
Ici aussi, il y a un problème

Les effets d'animation ?

@Clmnnt
Copy link
Contributor

Clmnnt commented Nov 30, 2024

trim.E3EEE0BF-5E44-4A45-9A9B-71F16F88F03E.MOV
Ici aussi, il y a un problème

Les effets d'animation ?

yep c'est ça

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants