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.

Sign up for GitHub

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

Jump to bottom

Mutiples fixes #294

Closed
wants to merge 36 commits into from
Closed

Mutiples fixes #294

wants to merge 36 commits into from

Conversation

Gabriel29306
Copy link
Contributor

@Gabriel29306 Gabriel29306 commented Oct 17, 2024 •
edited
Loading

🚀 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

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
fixe(Timetable): Salle widget cours PapillonApp#279
1bd4274
@Gabriel29306
fix(lessonDocument): use safe area insets
1aba2ab
@Gabriel29306
refactor(lessonDocument): Change animation to "slide_from_bottom"
33b8a35
@Gabriel29306
refactor(lessonDocument): new line between classes
16a0091
@Gabriel29306
fix(lessonDocument): Center subject's icon PapillonApp#256
9440bf1
@Gabriel29306
fix(expo): expo-font configuration
082ada8
@Gabriel29306
fix(typo): semiBold => semibold
57d895e
@Gabriel29306
chore(deps): upgrade deps
e680b1e
@Gabriel29306
refactor(extract_pronote_name.ts/upperFirst): Allow multiple familly …
0abac57 
…name.

Before: DE VINCI => De vinci
After: DE VINCI => De Vinci
@Gabriel29306
refactor: remove unecessary
3f1d072
@Gabriel29306
fix: link icon instead of file
621cd38
@Gabriel29306
fix(Widget/LastGrade): center icon + Round icon back
1c448db
@Gabriel29306
fix(Settings): fix "Soutenir Papillon" buttun overrided by "Quoi de n…
6fc7957 
…euf ?"
@Gabriel29306
refactor(SettingsTabs): Ensuring Home tab is in the correct position
2106541
@Gabriel29306
Merge branch 'main' of https://github.com/PapillonApp/Papillon into m…
5251ac4 
…utiples-fixes
@Gabriel29306
fix(news): infinite loading
5165f7b
@Gabriel29306
fix(Homework/Document): use safe area insets
a4c7ec8
@Gabriel29306
fix(Homework/Document): center subject icon
4f3a6d8
@Gabriel29306
feat: set animation from bottom
9912ef1
@Gabriel29306
feat(SettingsTabs): Automatic reorder tabs when enabled
7e26d29
@Gabriel29306
chore: upgrade deps
98ff9f0
@Gabriel29306
fix(NewsItem): use safe area insets
62a37db
@Gabriel29306
fix conflicts
ff805a1
@Gabriel29306
Copy link
Contributor Author

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

@Gabriel29306
Copy link
Contributor Author

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

@Gabriel29306
Copy link
Contributor Author

Je fais un build test et ça part en review

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 22, 2024

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

Sans commentaire...

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 22, 2024

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

@Gabriel29306
Copy link
Contributor Author

Node modifie quelle partie ?

image

Ah, bon c'est pas trop grave en vrai

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 22, 2024

Node modifie quelle partie ?

image

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
Copy link
Contributor Author

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
Copy link
Contributor Author

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

@tryon-dev
Copy link
Contributor

Hey je Check ça après

@yannouuuu yannouuuu added ✨ enhancement New feature or request 🪱 bug Something isn't working labels Oct 26, 2024
@Gabriel29306 Gabriel29306 mentioned this pull request Oct 28, 2024
Open
@Gabriel29306
Copy link
Contributor Author

@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
Copy link
Contributor

Kgeek33 commented Oct 28, 2024

@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
Kgeek33 approved these changes Oct 28, 2024
View reviewed changes
Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 30, 2024

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

@Gabriel29306
Copy link
Contributor Author

Il ne manque que @tryon-dev

@Gabriel29306
Copy link
Contributor Author

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
Copy link
Contributor

Kgeek33 commented Nov 3, 2024

Nonnnnnnn 😭😭😭😭

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 3, 2024

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

@Gabriel29306
Copy link
Contributor Author

Oui

This was referenced Nov 3, 2024
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kgeek33 Kgeek33 Kgeek33 approved these changes

@tryon-dev tryon-dev Awaiting requested review from tryon-dev tryon-dev is a code owner

Assignees
No one assigned
Labels
🪱 bug Something isn't working ✨ enhancement New feature or request
Projects
None yet
Milestone
No milestone
4 participants
@Gabriel29306 @Kgeek33 @tryon-dev @yannouuuu