fixs: Tous les bugs existants #325

Kgeek33 · 2024-11-03T14:43:37Z

Note

Cette PR a été faite dans sa grande majorité par @Gabriel29306 (#294 ), mais closed suite à l'enchaînement de conflit, VU QUE PERSONNE FAISAIT DES REVIEWS !!!

Elle est complétée avec les derniers bugs trouvés

🚀 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

Sur la PR #294

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

Sur ma PR en sup.

Correction d'un bug qui n'affichait rien quand on sélectionne une date sur l'emploi du temps
Optimisation du code
Homogénéité entre Apple et Android sur setShowDatePicker(false);
Comme à chaque maj de Papillon, les mises à jour de l'app sur Android pose problème, car la versionCode ne correspond pas à celle du Play Store
En ce moment, @LeGeek01 a mis une version code de "7211", alors je la passe à "7300"
Correction de l'affichage fonctionnant 1 fois sur 10 lorsqu'on clique sur un devoir
BONUS: ajout d'animations sur d'autres pages pour éviter ce genre de problème
Correction complète de la page d'accueil (marges, animations instables…)
Passage RenderHTML -> HTMLView pour arrêter les 3 avertissements de Node (fonctionnalité JavaScript déprécié)
Bump vers la version 7.4.0 + version code pour iOS modifiée pour homogénéité entre Apple et Android
Correction couleur et émoji qui ne changeait pas lors de la modification d'une matière (permet ainsi l'affichage d'une alerte en cas d'aucun emoji)
Généralisation de l'interface en passant de NativeText -> PapillonHeaderSelection sur Attendance.tsx (+ marginRight pour éviter que ça dépasse l'écran)
Fix de la redirection vers la semaine prochaine alors qu'on est en plein milieu de la semaine (Devoirs)
Fix des icônes qui ne s'affichait pas
Fix de la moyenne générale qui peut être vide sur le widget
Autres fix pour moins d'erreurs dans la console (exemple : FlatList)

Informations supplémentaires

Issues anciennes à closed

Sur la PR #294

Sur ma PR en sup.

Closed [BUG]: Affichage des devoirs / cours #310
Closes [BUG]: Utilisation du calendrier pour les cours #312
Closes [Bug]: Comportement bizarre quand sélection d'une date (Lessons) #232
Closes [Bug]: L'interface de l'accueil est instable (Android) #315
Closes [BUG]: Marges sur la mage d'accueil #311
Closes [Bug] problème de darkmode #318
Closes Bug Changement de couleur des matières #331
Closes [Bug] Le sélectionneur de dates ne marche pas #336
Closes [Bug]: Problème Menu écran d’accueil sur iPad #338
Closes [Bug]: Problème de devoir sur l'écran d'accueil #332
Closes Problème lors du retour au menu précédent #350
Closes Problème influence des notes sur la moyenne générale et moyenne de classe #358
Closes Couleur actualités #367

…name. Before: DE VINCI => De vinci After: DE VINCI => De Vinci

…euf ?"

…utiples-fixes

…ion.MagicNews`

Co-authored-by: 𝕂𝕪𝕝𝕚𝕒𝕟 <[email protected]>

imyanice · 2024-11-16T14:23:14Z

est ce que vous pouvez compartimenter vos prs et réécrire tout vos commits avec des descriptifs (fix 1, fix 2) et combiner ceux qui sont redondant (fix 1, fix 1 final, fix 1 final final, fix du fix 1).
merci!

Kgeek33 · 2024-11-16T14:28:52Z

mes commits (fix 1, 2...), c'était juste les erreurs typescript, c'est pour ça que j'ai pas détaillé
Sinon, je détaille oui

… constant design with the homework and timetable page" This reverts commit 9b560b0

Kgeek33 · 2024-11-16T14:31:24Z

@raphckrman j'ai corrigé le nom du compte illisible en mode clair

raphckrman · 2024-11-16T15:43:02Z

@raphckrman j'ai corrigé le nom du compte illisible en mode clair

Il y a une ombre bug maintenant

Kgeek33 · 2024-11-16T15:45:00Z

Eh mdrr
C'était mieux avant j'pense

raphckrman · 2024-11-16T15:59:35Z

Il y a un petit problème que j'ai vu en testant ton fork, elle consomme +770Mb de RAM pour une utilisation normal, alors que de base elle en consommait 320/400Mb (ce qui est déjà pas mal). Il y a une augmentation considérable de 200Mb en allant sur la page "Notes", et des augmentations aussi sur les pages "Cours" et "Devoirs"

ecnivtwelve · 2024-11-16T16:36:05Z

Il devient vraiment très très difficile de gérer cette PR, son comportement et ses objectifs au vu de sa taille et de son ancienneté pourtant facile a découper
Certains problèmes simples a régler auraient pu l'être depuis longtemps si il n'étaient pas couplés a de nombreux changements majeurs.

Ici, la PR risque de casser involontairement bon nombre de fonctionnalités.

Kgeek33 · 2024-11-16T16:41:50Z

Donc à la majorité, vous préférez que cette pr soit annulé pour que j'en fasse plusieurs mais plus légère ?

Kgeek33 · 2024-11-16T16:42:11Z

Une pr = un fichier modifié en gros ?

Vexcited · 2024-11-16T16:43:00Z

Donc à la majorité, vous préférez que cette pr soit annulé pour que j'en fasse plusieurs mais plus légère ?

Si possible, oui

tryon-dev · 2024-11-16T16:44:29Z

Donc à la majorité, vous préférez que cette pr soit annulé pour que j'en fasse plusieurs mais plus légère ?

Oui ca serait mieux

ecnivtwelve · 2024-11-16T16:46:03Z

Une pr = un fichier modifié en gros ?

Pas "un fichier", mais UN objectif ou au mieux une feature si pas éparpillée dans l'app (à noter fix de types et de formattage est un objectif INDÉPENDANT)

ecnivtwelve · 2024-11-16T16:47:53Z

Donc à la majorité, vous préférez que cette pr soit annulé pour que j'en fasse plusieurs mais plus légère ?

Si tu souhaites que l'ensemble des changements apparaissent rapidement, oui

Quelque chose de simple comme des types ou un simple changement dans l'UI ne prendrait que quelques minutes a merge
Un problème sur une PR ne bloque pas l'avancement d'une autre si elles sont bien indépendantes, contrairement à ici ou de nombreux changements pourtant prêts a passer en prod doivent attendre juste a cause d'un problème ailleurs
Permettre de tracer les changements et d'en discuter rendrait la gestion des discussions plus efficaces et BIEN plus rapide

Kgeek33 · 2024-11-16T16:47:57Z

OK ok, je vais faire ça, je close quand j'aurai totalement terminé
Déso de cette grosse pr 😅

ecnivtwelve · 2024-11-16T16:48:52Z

OK ok, je vais faire ça, je close quand j'aurai totalement terminé Déso de cette grosse pr 😅

On fera l'effort de les merge sous peu, d'autant plus que tes fix sont plutôt intéressants !

Kgeek33 · 2024-11-16T16:51:40Z

OK ok, je vais faire ça, je close quand j'aurai totalement terminé Déso de cette grosse pr 😅

On fera l'effort de les merge sous peu, d'autant plus que tes fix sont plutôt intéressants !

Haha merci.
Comme la version 7.5 a été publié sur les stores bêta (jspas pour Apple), est-ce qu'on bump vers une 7.6 ou 7.5.1 ?

tryon-dev · 2024-11-16T16:56:37Z

nop justement c'est une beta pour l'instant qui n'est pas finalisé

tryon-dev · 2024-11-16T16:56:56Z

il reste encore plusieurs build pour sortir sur les stores

Kgeek33 · 2024-11-20T18:25:16Z

Ok tous les fixs ont été séparés dans plusieurs pr (sauf l'affichage centré des émojis, en cours de travail), cette pr est désormais closed

Gabriel29306 and others added 30 commits October 10, 2024 19:28

fixe(Timetable): Salle widget cours PapillonApp#279

1bd4274

fix(lessonDocument): use safe area insets

1aba2ab

refactor(lessonDocument): Change animation to "slide_from_bottom"

33b8a35

refactor(lessonDocument): new line between classes

16a0091

fix(lessonDocument): Center subject's icon PapillonApp#256

9440bf1

fix(expo): expo-font configuration

082ada8

fix(typo): semiBold => semibold

57d895e

chore(deps): upgrade deps

e680b1e

refactor(extract_pronote_name.ts/upperFirst): Allow multiple familly …

0abac57

…name. Before: DE VINCI => De vinci After: DE VINCI => De Vinci

refactor: remove unecessary

3f1d072

fix: link icon instead of file

621cd38

fix(Widget/LastGrade): center icon + Round icon back

1c448db

fix(Settings): fix "Soutenir Papillon" buttun overrided by "Quoi de n…

6fc7957

…euf ?"

refactor(SettingsTabs): Ensuring Home tab is in the correct position

2106541

Merge branch 'main' of https://github.com/PapillonApp/Papillon into m…

5251ac4

…utiples-fixes

fix(news): infinite loading

5165f7b

fix(Homework/Document): use safe area insets

a4c7ec8

fix(Homework/Document): center subject icon

4f3a6d8

feat: set animation from bottom

9912ef1

feat(SettingsTabs): Automatic reorder tabs when enabled

7e26d29

chore: upgrade deps

98ff9f0

fix(NewsItem): use safe area insets

62a37db

fix conflicts

ff805a1

fixs

2b95fb4

fix: use safe area insets

2c65edc

fix: use safe area insets

d22d096

refactor: Change donation button for Android only

dd50eee

fix(GradeItem): center subject's emoji

ed8bad8

fix: account.personalization?.magicEnabled => `account.personalizat…

2e7a2a6

…ion.MagicNews`

Perf: Remove redondant instruction

5edcdb7

Co-authored-by: 𝕂𝕪𝕝𝕚𝕒𝕟 <[email protected]>

Revert "!feat(AccountSwitcher): AccountSwitcher is redisigned to keep…

5cc3a1b

… constant design with the homework and timetable page" This reverts commit 9b560b0

fix affichage version d'expo non fonctionnel

3bd09df

ecnivtwelve mentioned this pull request Nov 16, 2024

Feat(News): intégration "J'ai pris connaissance..." + changes #281

Merged

9 tasks

Kgeek33 closed this Nov 20, 2024

Kgeek33 deleted the fix/allBug branch November 20, 2024 18:25

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fixs: Tous les bugs existants #325

fixs: Tous les bugs existants #325

Kgeek33 commented Nov 3, 2024 •

edited

Loading

imyanice commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

raphckrman commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

raphckrman commented Nov 16, 2024

ecnivtwelve commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

Vexcited commented Nov 16, 2024

tryon-dev commented Nov 16, 2024

ecnivtwelve commented Nov 16, 2024

ecnivtwelve commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

ecnivtwelve commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

tryon-dev commented Nov 16, 2024

tryon-dev commented Nov 16, 2024

Kgeek33 commented Nov 20, 2024

fixs: Tous les bugs existants #325

fixs: Tous les bugs existants #325

Conversation

Kgeek33 commented Nov 3, 2024 • edited Loading

🚀 Nouvelle Pull Request

Informations importantes

Checklist d'avant pull request

Changelogs proposés

Sur la PR #294

Sur ma PR en sup.

Informations supplémentaires

Issues anciennes à closed

Sur la PR #294

Sur ma PR en sup.

imyanice commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

raphckrman commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

raphckrman commented Nov 16, 2024

ecnivtwelve commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

Vexcited commented Nov 16, 2024

tryon-dev commented Nov 16, 2024

ecnivtwelve commented Nov 16, 2024

ecnivtwelve commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

ecnivtwelve commented Nov 16, 2024

Kgeek33 commented Nov 16, 2024

tryon-dev commented Nov 16, 2024

tryon-dev commented Nov 16, 2024

Kgeek33 commented Nov 20, 2024

Kgeek33 commented Nov 3, 2024 •

edited

Loading