Gabriel29306 commented 1 month ago

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

[x] Vous avez testé de build le projet avec vos modifications et ce build a réussi
[x] Vous respectez les conventions de codage et de nommage du projet
[x] Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
[x] Cette pull request n'est pas un duplicata d'une autre
[x] Cette pull request est prête à être revue (review) et fusionnée (merge)
[x] Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
[x] Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
[x] Les détails des changements ont été décrits ci-dessous
[x] 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

fix: Centrage des icones pour les cours, les devoirs, widget LastGrade fix: majuscules pour les noms de famille composés fix: nom de police: semiBold => semibold fix: icone de pièce jointe pour les communications fix: bouton soutenir Papillon dans les paramètres fix: chargement infini des actualités fix: utilisation des safe area insets pour les views ayant un style de modale refactor: Changement de l'animation pour les views ayant un style de modale refactor: réarrangement des onglets dans SettingsTabs refactor: configuration d'expo-font refactor: nouvelles lignes entre les salles de classes refactor: bouton de donation en Android only chore: Mise à jour des dépendances

Informations supplémentaires

fix #279 fix #256 fix #233 fix #317 fix peut-être d'autres issues, maybe

Gabriel29306 commented 1 month ago

Faut juste que je teste le tout puis je la met en ready

Gabriel29306 commented 1 month ago

Bon, il n'y a plus d'animations pour les NewsItems, c'est pas trop grave

Gabriel29306 commented 1 month ago

Je fais un build test et ça part en review

Gabriel29306 commented 1 month ago

Pour les actualités je ne sais pas d'où cela peut venir, c'est à partir du moment où j'ai fusionné pour fix les conflits que ça a foiré.
Pour les emoji je m'en suis occupé, c'était rapide
Pour les modales il faudrait revoir le fonctionnement pour faire comme sur Apple. (J'ai le même réflexe)

Kgeek33 commented 1 month ago

Oui moi aussi, depuis l'intégration d'école directe et d'autres modifs comme les devoirs intelligents, la page d'actualité est remplie de conflits et ça me casse ma pr #281
Pour les émojis, il y a un marginTop trop élevé, qui fait que l'émoji est un peu coupé (notamment l'émoji 🏀)
J'ai trouvé ceci pour faire glisser une modal : https://github.com/jefelewis/react-native-slide-modal

Gabriel29306 commented 1 month ago

Normalement j'ai réglé les emojis comme il faut, je vais vérifier que mes changements n'ont pas été overrides par le merge

Kgeek33 commented 1 month ago

J'ai oublié de préciser, c'est sur la page d'accueil, la liste des dernières notes que ce comportement a lieu

Gabriel29306 commented 1 month ago

C'est bon, c'est fait

Gabriel29306 commented 1 month ago

Pour les conflits faut que je passe par un ordi

Kgeek33 commented 1 month ago

Je teste et je te dis si ça fonctionne

Kgeek33 commented 1 month ago

T'en profiteras pour mettre à jour le fichier package-lock.json quand tu résoudras tes conflits 😉

Gabriel29306 commented 1 month ago

Node modifie quelle partie ?

Gabriel29306 commented 1 month ago

LGTM 👍 Le package-lock.json n'est tjrs pas à jour, node me l'a encore modifié 😅

Pourquoi vous passez votre temps à insulter les mères sur ce repo ?

Hein ?

Kgeek33 commented 1 month ago

Node modifie quelle partie ?

Kgeek33 commented 1 month ago

Pourquoi vous passez votre temps à insulter les mères sur ce repo ?

Sans commentaire...

Kgeek33 commented 1 month ago

laisse tomber le react-native-slide-modal, j'suis en train de tester

Gabriel29306 commented 1 month ago

Node modifie quelle partie ?

Ah, bon c'est pas trop grave en vrai

Kgeek33 commented 1 month ago

Node modifie quelle partie ?

Ah, bon c'est pas trop grave en vrai

Oui ça va. Bon j'suis en train de tester pour faire glisser les modals sur Android. Si ça marche, j'te montre pour que tu l'intègres dans ta pr 😉

Gabriel29306 commented 1 month ago

Oui ça va. Bon j'suis en train de tester pour faire glisser les modals sur Android. Si ça marche, j'te montre pour que tu l'intègres dans ta pr 😉

C'est gentil mais non, le but de cet PR est de fixer quelques problèmes, pas d'ajouter des features

Gabriel29306 commented 1 month ago

@tryon-dev ce serait cool d'avoir ta review 😁

tryon-dev commented 1 month ago

Hey je Check ça après

Gabriel29306 commented 1 month ago

@tryon-dev si tu pouvais review, ce serait cool. C'est un peu casse pied de devoir résoudre régulièrement les conflits 😅

Kgeek33 commented 1 month ago

@tryon-dev si tu pouvais review, ce serait cool. C'est un peu casse pied de devoir résoudre régulièrement les conflits 😅

Je suis tellement d'accord 🤣 Je review après le build de mon site ;)

Kgeek33 commented 4 weeks ago

SVP, acceptez cette pr !! Ça évitera les checks failed sur les nouvelles/actuelles pr, et corrigera bcp de bugs

Gabriel29306 commented 4 weeks ago

Il ne manque que @tryon-dev

Gabriel29306 commented 3 weeks ago

La PR a trop traînée et j'en ai marre de devoir résoudre les conflits à chaque autre PR acceptée. Flûte !

Kgeek33 commented 3 weeks ago

Nonnnnnnn 😭😭😭😭

Kgeek33 commented 3 weeks ago

J'peux te "l'emprunter"? 😂😂

Gabriel29306 commented 3 weeks ago

Oui

PapillonApp / Papillon

Mutiples fixes #294

🚀 Nouvelle Pull Request

Informations importantes

Checklist d'avant pull request

Changelogs proposés

Informations supplémentaires