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 Mar 25, 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...

Tu t'assures avoir respecté les points suivants (en rajoutant un x dans les crochets) :

  • 📲 J'ai testé l'application via Expo Go et/ou Build et elle fonctionne correctement
  • 🗣️ J'utilise le langage informel (tutoiement)
  • 📃 Cette PR n'est pas un duplicata et est prête à être review et merge
  • ❌ Je n'ai pas fait d'erreurs TypeScript et ESLint dans le code
  • 📝 J'ai fait une description des changement effectués ci-dessous

Résumé des changements effectués

  • Utilisation de <PapillonModernHeader> sur toutes les pages

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

undefined (surtout pour Android)

Issue(s) en rapport

Note

Cette section te permet de référencer des issues en lien avec ta PR. Pour que les issues soient fermées automatiquement au merge, utilise les "closing keywords" de Github (exemples ci-dessous).

S'il y en a plusieurs, continuer à les lister

  • Closed #(le numéro de l'issue)

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Mar 25, 2025

@ecnivtwelve déso pour ma précédente pr 😅
là, j'ai fait simple !

@NathanBnm
Copy link
Contributor

@Kgeek33 est-ce que le travail que tu as fait sur cette PR permettra de résoudre #850 ?
J'ai lu ton commentaire https://github.com/PapillonApp/Papillon/pull/856#issuecomment-2745322670 et je suis du même avis

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Mar 26, 2025

@Kgeek33 est-ce que le travail que tu as fait sur cette PR permettra de résoudre #850 ? J'ai lu ton commentaire #856 (comment) et je suis du même avis

non ça ne permet pas de résoudre ce problème. cependant, si je le fais sur une autre pr, cela réduirait énormément de fonctionnalités sur iOS et faudrait refaire en parti le routeur

@Kgeek33 Kgeek33 closed this Apr 30, 2025
@Kgeek33 Kgeek33 deleted the feat/tabs branch April 30, 2025 23:53
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.

[Bug]: Certains onglets n’ont pas de croix
2 participants