Update SettingsAbout.tsx

[TinAD17tin]

Ajustement de la méthode de récupération de version d’expo, car sinon, ça n’était pas afficher dans l’application a cause de l’utilisation de ~ dans package.json alors que ^ était utilisé ici.

🚀 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

Décrivez les modifications que vous avez effectuées.

Passage de "^" à "~" lors de la récupération de la version d’expo.
Utilisation du même code que pour la récupération de RN permettant d’être sûr que ça marchera

Informations supplémentaires

Ajoutez ici toute information supplémentaire si nécessaire.

Sur la 7.7, la version d’Expo n’est pas/plus affichée

[TinAD17tin]

Je n’ai pas la possibilité de build + testé donc si quelqun peut le faire pour moi, ce serait gentil 🙂.
C’est ma première contribution

[JyhuKo]

ca marcher pas pour moi

[JyhuKo]

essay de replace par ca RN : {PackageJSON.dependencies["react-native"].split("^")[1]} | Expo : {(PackageJSON.devDependencies.expo || PackageJSON.dependencies.expo).split("^")[1]}

[TinAD17tin]

Nan parce que du coup le but c’est de ne pas utilisé ^ mais ~ maintenant, parce que dans package.json c’est ~ donc le split il marchait pas @JyhuKo

[JyhuKo]

("^")[1]}

sur mon expo go ca marche pourtant

[TinAD17tin]

7.7 ça marche pas

[TinAD17tin]

Et dans https://github.com/PapillonApp/Papillon/blob/main/package.json
C’est à chaque fois "expo": "~ et non ^ que ce soit dans devdependcies ou dans depencies tous court

[JyhuKo]

my badd mon package json etait pas a jour , du coup ca marche bien

[TinAD17tin]

Parfait !

[Kgeek33]

Cool, t'as fait le même fix que moi sur #571 (mais au moins la review sera plus rapide)

Du coup, j'ai fait comme ceci sur ma PR, c'est plus simple que le replace et ça gère tous les cas de version d'expo ;)

[TinAD17tin]

Normalement la on est bon.
Ça récupère que ce soit ^ ou ~ ou rien.

[Kgeek33]

@JyhuKo tu pourras tester ? Je savais pas qu'un split comme ceci pouvait marcher 😂

[TinAD17tin]

Normalement si je suis pas sûr de l’info

[Kgeek33]

Ben ça a l'air bon dc 😂