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.

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

[Bug]: Actualités qui chargent pas sur ED #300

Open
okayhappex opened this issue Oct 22, 2024 · 17 comments
Open

[Bug]: Actualités qui chargent pas sur ED #300

okayhappex opened this issue Oct 22, 2024 · 17 comments

Comments

@okayhappex
Copy link
Contributor

Description du bug

En gros quand je me connecte sur une instance ED et sur j'essaye de charger les actualités l'écran se secoue comme pas possible mais les actualités s'affichent

Sur le screen que je vais envoyer on voit rien parce que je suis hors période scolaire mais j'ai découvert le bug la semaine dernière et les actualités apparaissaient et disparaissaient ultra rapidement

Étapes à reproduire

  1. Se connecter sur ED
  2. Aller dans l'onglet "actualités"

Comportement attendu

Les actualités devraient s'afficher... Sans disparaître

Appareil

Honor 70 Lite

Version du système d`exploitation

Android 14

Version

7.2.1

Environnement

Prébuild de développement (iOS/Android)

Source de l'application

Stores (Play Store / App Store)

Service scolaire

🟦 EcoleDirecte

Captures d'écran / vidéo

SVID_20241022_151000_1.mp4

Contexte supplémentaire

J'ai essayé sur le build de @Evorsiooo aujourd'hui et sur le repository officiel (celui-ci) la semaine dernière via Expo Go

https://github.com/Evorsiooo/Papillon/releases/tag/android

@okayhappex
Copy link
Contributor Author

La vidéo date d'aujourd'hui mais le comportement observé est exactement le même sur les deux instances

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 22, 2024

Tu peux envoyer les logs stp ?

  • Paramètres
  • À propos de Papillon
  • Tout en bas, clique sur "Version des dépendances"

@okayhappex
Copy link
Contributor Author

Désolé je trouve rien datant d'après le 2 octobre
Screenshot_20241022_161423_xyz_getpapillon_app_MainActivity

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 22, 2024

ah bizarre. je peux essayer de t'aider, mais j'suis pas sur Ecole Directe, donc ça va être un peu difficile 😅
T'as déjà reçu des actualités ?

@okayhappex
Copy link
Contributor Author

La semaine dernière oui

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 22, 2024

Alors

  • Soit ce sont les useState qui sont mal gérées avec École Directe
  • Soit la récupération des données n'est pas fonctionnelle

Le 1er point étant le plus probable

@okayhappex
Copy link
Contributor Author

Je viens d'avoir une actu et elle se recharge toutes les 3 secondes. En revanche seul le prénom de la personne et la date relative de publication (il y a un jour) s'affichent

Par un souci de confidentialité je peux pas trop envoyer de vidéo

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 24, 2024

Donc logiquement, ça veut dire que c'est ici que le code pose problème

https://github.com/PapillonApp/Papillon/blob/main/src%2Fviews%2Faccount%2FNews%2FNews.tsx#L45-L48

Ça veut dire que account.instance change tout le temps et du coup, le useEffect appelle la fonction fetchData à l'infini

Indice : dans la vidéo, on voit le truc de chargement (le spinner, jspas si ça s'appelle comme ça) qui s'affiche, se cache et se réaffiche et ainsi de suite. Et seule la fonction fetchData permet de gérer l'affichage du spinner

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 24, 2024

Encore une fois, gros pavé pour faire comprendre ce que j'en pense

Mais je trouve ça bizarre quand même que account.instance change tout le temps

@imyanice
Copy link
Contributor

je veux bien aider, mais si on ne m'envoie pas la réponse d'écoledirecte je ne peux rien faire....

inspecter l'élément, réseau et vous m'envoyez la requête des actualités.

CleanShot 2024-10-28 at 12 22 11@2x

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 8, 2024

est-ce toujours le cas @okayhappex ?

@okayhappex
Copy link
Contributor Author

Oui et pour la 7.3 le build a quand même beaucoup plus de bugs que les build qui ont été faits par différents membres

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 8, 2024

OK mince
Alors peux-tu tester ma pr #325 et me dire si tu rencontres encore le problème
Comme ça, j'essaie de corriger ça avant d'accepter les review

@okayhappex
Copy link
Contributor Author

Ça va être galère vu que je suis dans le car jusqu'à minuit et je dors direct en arrivant mais si t'as un build sur toi je veux bien stp

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 8, 2024

OK pas de problème, j'tenvoie dans quelques minutes, le temps que je rentre chez moi

@imyanice
Copy link
Contributor

imyanice commented Nov 8, 2024

Je vais push des fix ce week-end pour résoudre les nouveaux bugs.

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 8, 2024

ok très bien
@okayhappex je pourrai pas builder, car les devoirs posent énormément de problème maintenant 😭

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

No branches or pull requests

3 participants