Skip to content
This repository was archived by the owner on Aug 30, 2025. It is now read-only.

Conversation

Kgeek33
Copy link
Contributor

@Kgeek33 Kgeek33 commented Jan 28, 2025

✨ Nouvelle Pull Request

Merci de contribuer à l'amélioration de Papillon !

Tu te poses des questions sur les pull requests (PR) ? Une documentation a spécialement été crée ici => https://gitbook.getpapillon.xyz/organisation/outils-internes/github

Avant toute chose...

Pour nous aider à tester ta PR, merci de cocher une des cases suivantes (en rajoutant un x dans les crochets) :

Type de pull request :

  • Breaking change (des modifications avec un impact sur les fonctionnalités actuelles)
    • (plus tard) J'ai fait une build de Papillon pour m'assurer que je n'ai rien cassé
  • Feat (ajoute une amélioration/nouveauté)
    • Je m'assure que j'utilise le langage informel (tutoiement)
  • Fix (permet de corriger un bug)
  • Chore (des modifications en dehors du dossier src)
  • Styles (change/ajoute du style)

Résumé des changements effectués

Note

Même si RN 0.77 a été récemment publié, Expo 52 s'appuie sur RN 0.76 !

Papillon est désormais compatible avec Expo 52, donc RN 0.76 !!
Et j'ai intégré npm-check-updates qui permet de mettre à jour les dépendances (à l'exeption de 3) vers leur dernière version

Capture(s) d'écran (pour rendre le test de ta PR rapide)

bientôt

Ce qui est prévu avant qu'elle soit prête

Note

Cela concerne uniquement Expo Go, sera actualisé au moment des tests de build

  • Feat: Refaire le splash screen car fonctionnement différent sur Expo 52
  • Fix: Glissade des modals impossible (iOS) => en profiter pour mettre les modales 100% glissables pour Android
  • Fix: Scroll des pages impossible jusqu'à la fin
  • Fix: Style des modals (modif des matières) extrêmement moche (Android)
  • Fix: Graphique des notes inutilisable (Android) => faire un composant si nécessaire
  • (pas sûr encore) Feat: AlertProvider sur Android ET iOS + supprimer toutes les Alert.Alert
  • Feat: Créer workflow pour faire un qr code pour tester les pr (Utiliser la base de la pr 📱🚀 GitHub : Preview à l'aide d'ExpoGo lors d'une PR (Sans EAS) #260)

Note

Le temps que les points ci-dessus soient faits, commencez à review ! Cela m'aidera à corriger rapidement les bugs que je n'aurai pas vu

A l'équipe Papillon (par rapport à leur message sur #370) :

@JyhuKo
Copy link
Contributor

JyhuKo commented Jan 28, 2025

le retourrrr

@Clmnnt
Copy link
Contributor

Clmnnt commented Jan 29, 2025

COMEBAAACK

@Gabriel29306
Copy link
Contributor

🤩

@julesjung
Copy link
Contributor

🥹

@LeGeek01
Copy link
Contributor

28 commits et encore des conflits ptdrrrr

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Jan 29, 2025

28 commits et encore des conflits ptdrrrr

Ah mais c'est ça l'avantage de ce type de pr !
Tu fais une p'tite modif sur un fichier et t'as direct un conflit 😂💀

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Jan 29, 2025

Le check est ok maintenant !

@godetremy
Copy link
Member

Note

Cette review est le résultat de la concertation de toute l'équipe Papillon.

Papillon ne serra pas disponible sur Expo 52 sans rewrite.

Pourquoi ?

Transformer Papillon vers Expo52 pose beaucoup de soucis sur la stabilité de celle-ci. Afin d'être certains de la stabilité, il faudrait faire une beta a grande échelle, ce qui n'ai pas faisable.

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

Successfully merging this pull request may close these issues.

8 participants