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

COM-2929: expo sdk 44 #478

Merged
merged 8 commits into from Jul 29, 2022
Merged

COM-2929: expo sdk 44 #478

merged 8 commits into from Jul 29, 2022

Conversation

ulysseferreira
Copy link
Collaborator

@ulysseferreira ulysseferreira commented Jul 27, 2022

Faire un npm i -g expo-cli
il peut être nécessaire de désinstaller et réinstaller Expo sur les terminaux pour avoir la dernière version d'Expo Go

breaking changes : https://blog.expo.dev/expo-sdk-44-4c4b8306584a
changelogs : https://github.com/expo/expo/blob/master/CHANGELOG.md#4400--2021-12-03

React 17 JSX transform now enabled by default : il n'est plus nécessaire d'importer React sur chaque fichier (retiré avec un script fourni par React : https://reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html)

jest-expo no longer depends directly on jest : les deux packages sont désormais distinct, d'où la modification du chemin d'accès pour les tests

Versions majeurs MAJ :
Expo 43 => 44

expo-constants 12.1.3 => 13.0.1

  • Dropped support for iOS 11.0

expo-firebase-analytics 5.0.3 => 6.0.0

  • Dropped support for iOS 11.0

expo-image-picker 11.0.3 => 12.0.1

  • Dropped support for iOS 11.0

expo-linear-gradient 10.0.3 => 11.0.3

  • Dropped support for iOS 11.0

react-native-gesture-handler 1.10.2 => 2.1.0

babel-preset-expo 8.5.1 => 9.1.0

  • Add lazyImports option

jest-expo 43 => 44

  • Removed mockPlatformIOS(), mockPlatformAndroid(), mockPlatformWeb(), describeCrossPlatform() in favor of platform specific presets like jest-expo/universal

Autre paquets maj
typescript :

  • il était downgrade par Expo 44 et c'est pour ça que les tests eslint cassait, je l'ai donc remis à sa version d'avant + petite MAJ qui n'entraine pas de breaking changes (essentiellement des fixes de bugs)

axios :

  • passage à la version 0.27 qui fixe l'envoi de FormData. La version qu'on avait d'axios était buggé et ne permettait plus d'envoyer des objets FormData d'où le fait qu'on ne pouvait plus envoyé de photo. C'est corrigé dans cette version. (version en place sur la webapp c'est pourquoi ça fonctionnait toujours de ce côté)
    https://github.com/axios/axios/releases

react-native-draggable-flatlist :

  • le package cassait, je l'ai donc mis à jour. J'ai dû faire quelques autres modifs car en l'état la carte ne s'affichait pas bien (propositions en haut de la carte au lieu d'être en bas + footer coupé par le bas de l'écran après validation). J'ai repris le même schéma que les autres cartes (QCU, QCM notamment) avec la scrollview

@cmolle
Copy link
Contributor

cmolle commented Jul 28, 2022

typescript : il était downgrade par Expo 44 et c'est pour ça que les tests eslint cassait, je l'ai donc remis à sa version d'avant

C : Sais-tu pourquoi il avait ete downgrade pr expo 44 ?

U : expo a toujours eu une version en dessous de celle qu'on utilisait, on avait toujours un peu d'avance concernant typescript, notamment il me semble parce qu'on avait besoin d'une version plus avancée pour certains packages

Chloe : et du coup c'est pas genant qu'on le downgrade maintenant ? On a plus besoin d'une version + avancée ?

U : la gestion était différente avant vu que jest et jest-expo était liés. jest-expo utilisait la version 26 de jest (cf en dessous)
Capture d’écran 2022-07-28 à 13 49 23
on avait la 27 d'installer indépendamment mais je suis pas sûr que c'était utile

.eslintrc.js Show resolved Hide resolved
package.json Show resolved Hide resolved
package.json Show resolved Hide resolved
src/context/createDataContext.tsx Outdated Show resolved Hide resolved
@cmolle
Copy link
Contributor

cmolle commented Jul 28, 2022

Je n'arrive pas a lancer l'app.
J'ai ces erreurs :

Error: Requiring module "node_modules/react-native-reanimated/src/Animated.js", which threw an exception: Error: >Reanimated 2 failed to create a worklet, maybe you forgot to add Reanimated's babel plugin?


`Invariant Violation: "main" has not been registered. This can happen if:
* Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current >project.
* A module failed to load due to an error and AppRegistry.registerComponent wasn't called.

tu as rm rf puis coupé et relancé Expo ?

Chloé : Oui, j'ai meme desinstaller et reinstalle expo sur mon telephone + enlever le cache d'expo

U : j'avais ce soucis aussi au début et j'avais appliqué ce fix software-mansion/react-native-reanimated#1875 mais je l'avais retiré à la fin parce que j'avais l'impression qu'au final ça fonctionnait sans. Je l'ai remis du coup

@cmolle
Copy link
Contributor

cmolle commented Jul 28, 2022

Dropped support for iOS 11.0

C : As-tu regardé si on avait beaucoup d'utilisateurs qui sont sous ios 11 ? On peut le voir dans l'appstoreconnect normalement

U : visiblement aucun utilisateur sous iOS 11 (+ ils avaient commencé à lâcher le support pour ios11 sous SDK 43)

@ulysseferreira ulysseferreira self-assigned this Jul 29, 2022
@cmolle
Copy link
Contributor

cmolle commented Jul 29, 2022

~~J'ai des problemes sur l'utilisation des balises JSX 'View' cannot be used as a JSX component. Its instance type 'View' is not a valid JSX element. Est-ce qu'il faut faire une manip particuliere ? ~~

Finalement c'est bon, j'avais supprime le package-lock.

cmolle
cmolle previously approved these changes Jul 29, 2022
@cmolle cmolle dismissed their stale review July 29, 2022 13:31

Probleme photo

manonpalin
manonpalin previously approved these changes Jul 29, 2022
@manonpalin manonpalin dismissed their stale review July 29, 2022 14:00

Erreur sur les cards

@manonpalin
Copy link
Collaborator

manonpalin commented Jul 29, 2022

Quand je lance une activité puis que j'arrive sur une QCM et QCU j'ai l'erreur suivante : VirtualizedLists should never be nested inside plain ScrollViews with the same orientation because it can break windowing and other functionality - use another VirtualizedList-backed container instead

Pb connu : cf ticket

@manonpalin manonpalin merged commit ec24cf6 into dev Jul 29, 2022
@manonpalin manonpalin deleted the COM-2929 branch July 29, 2022 15:18
@sophiemoustard sophiemoustard added the dependencies Pull requests that update a dependency file label Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants