[Bug]: Notes qui chargent indéfiniment #301

okayhappex · 2024-10-22T13:30:34Z

Description du bug

Sur une instance ED lorsqu'on ouvre l'onglet Notes elles se mettent à charger indéfiniment

Étapes à reproduire

Se connecter sur une instance ED
Ouvrir l'onglet Notes
Regarder en haut à gauche (attention c'est hypnotique)

Comportement attendu

Notre 20/20 de moyenne devrait s'afficher en grand

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

Github

Service scolaire

🟦 EcoleDirecte

Captures d'écran / vidéo

SVID_20241022_151016_1.mp4

Contexte supplémentaire

Le bug est connu sur Discord depuis quelques jours mais pas répertorié sur Github
La vidéo provient du build de @Evorsiol mais le même comportement est observé sur ce repo via Expo Go

Kgeek33 · 2024-10-22T13:36:31Z

Tu peux envoyer les logs stp ?

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

okayhappex · 2024-10-22T14:15:31Z

Rien qui date d'après le 2 octobre

Kgeek33 · 2024-10-22T19:15:24Z

T'as déjà eu des notes ?

okayhappex · 2024-10-22T22:04:54Z

5 ou 6 ouais

Kgeek33 · 2024-10-22T22:09:14Z

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

Kgeek33 · 2024-10-22T22:09:56Z

Mdrr je fais que des copié/collé de #300 😂

okayhappex · 2024-10-22T22:39:24Z

Depuis taleur j'ai remarqué ouais 😂

Evorsiooo · 2024-10-23T20:32:18Z

Si je me souvient bien certaines personnes arrivaient a voir les notes (dont @imyanice je crois)
Une hypothèse qui avait était émise était donc que Papillon avait besoin de certaines infos (genre la note mini ou maxi d'un DS) mais que l'établisement avait fait le choix de ne pas donner l'information. Mais bon j'ai pas encore vérifé le code donc je sais pas 😆

Gabriel29306 · 2024-10-23T20:35:36Z

Ah, donc tu penses que Papillon est grincheux quand il n'a pas tout ce qu'il veut ?

Kgeek33 · 2024-10-23T20:48:37Z

Non. J'ai regardé le code et voilà ce que j'en pense.

Les variables suivantes permettent de récupérer les périodes et celle par défaut (en ce moment, c le trimestre 1/semestre 1)
https://github.com/PapillonApp/Papillon/blob/main/src%2Fviews%2Faccount%2FGrades%2FGrades.tsx#L32-L33
Une des 2 variables qui est définie ci-dessus est converti en useMemo
https://github.com/PapillonApp/Papillon/blob/main/src%2Fviews%2Faccount%2FGrades%2FGrades.tsx#L38

Mais il y a un problème dans tout ça, qui amène au 3ème point ci-dessous.

Le useEffect suivant permet de gérer le chargement de la page
https://github.com/PapillonApp/Papillon/blob/main/src%2Fviews%2Faccount%2FGrades%2FGrades.tsx#L62-L74

Mais la vidéo montre un chargement un chargement infini, donc le useEffect qui ne fonctionne pas.
Et j'en reviens à ma théorie : Papillon n'arrive pas à récupérer la période actuelle, d'où le chargement infini -> https://github.com/PapillonApp/Papillon/blob/main/src%2Fviews%2Faccount%2FGrades%2FGrades.tsx#L63

Kgeek33 · 2024-10-23T20:49:20Z

C'est très long ce que j'ai écrit, mais j'ai essayé d'être le plus clair possible 😅

Kgeek33 · 2024-10-23T20:50:17Z

Et en plus, regardez l'énorme indice dans la video :

La période actuelle n'est pas récupérée

okayhappex · 2024-10-24T03:56:22Z

Et en plus, regardez l'énorme indice dans la video :

La période actuelle n'est pas récupérée

Pas mal Inspecteur Geekjet 😌👌

imyanice · 2024-10-24T05:09:31Z

Compliqué à régler tout ça, vu que ça marche chez moi 😅.

Me faudrait le notes.awp obtenable dans le menu réseau de inspecter l'élément pour faire quelque chose

saumon-brule · 2024-10-27T21:56:12Z

Au risque d'être impertinent si vous saviez déjà Ecole directe envoi pas que les périodes dans les notes, la réponse contient aussi des relevés ou d'autres trucs, je sais pas comment c'est géré ni si vous parlez de ces périodes du tout et peut-être que le problème à rien à voir mais bon on sait jamais ca pourrait poser problème 😅

Evorsiooo · 2024-10-27T22:22:51Z

Compliqué à régler tout ça, vu que ça marche chez moi 😅.

Me faudrait le notes.awp obtenable dans le menu réseau de inspecter l'élément pour faire quelque chose

D'ailleurs je l'avais envoyé dans tes DMs il y a une bonne semaine mais tu m'a jamais répondu...

imyanice · 2024-10-28T07:52:56Z

Compliqué à régler tout ça, vu que ça marche chez moi 😅.
Me faudrait le notes.awp obtenable dans le menu réseau de inspecter l'élément pour faire quelque chose

D'ailleurs je l'avais envoyé dans tes DMs il y a une bonne semaine mais tu m'a jamais répondu...

@Evorsiooo j'étais en vacances! renvoie moi un dm et je regarde ça :)

Au risque d'être impertinent si vous saviez déjà Ecole directe envoi pas que les périodes dans les notes, la réponse contient aussi des relevés ou d'autres trucs, je sais pas comment c'est géré ni si vous parlez de ces périodes du tout et peut-être que le problème à rien à voir mais bon on sait jamais ca pourrait poser problème 😅

@saumon-brule je ne vois pas de quoi tu parles, si tu aurais un exemple, je suis preneur

saumon-brule · 2024-10-28T09:11:25Z

@imyanice la gestion des notes de ED est un peu bizarre (de toutes leurs API de ED il me semble que c'est vraiment la pire à parse)
La réponse ressemble en gros à ça :

{
  "code": 200,
  "token": "token",
  "host": "HTTP46",
  "data": {
    "periodes": [
      {
        "idPeriode": "A001",
        "codePeriode": "A001",
        "periode": "1er Trimestre",
        "annuel": false,
        "examenBlanc": false,
        "cloture": true,
        "ensembleMatieres": {
          "disciplines": [
            // une liste des matières de ce trimestre/semestre
            ]
        }
      },
      {
        "idPeriode": "A001R001",
        "codePeriode": "A001R001",
        "periode": "Relev\u00e9 ",
        "ensembleMatieres": {
          "disciplines": [
            // liste des matière ...
          ]
        }
      },
      {
        "idPeriode": "A002",
        "codePeriode": "A002",
        "periode": "2\u00e8me Trimestre",
        "ensembleMatieres": {
          "disciplines": [
            // ...
          ]
        }
      },
      {
        "idPeriode": "A003",
        "codePeriode": "A003",
        "periode": "3\u00e8me Trimestre",
        "ensembleMatieres": {
          "disciplines": [
            // ...
          ]
        }
      },
      {
        "idPeriode": "A999Z",
        "codePeriode": "A999Z",
        "periode": "Ann\u00e9e",
        "ensembleMatieres": {
          "disciplines": [
            // ...
          ]
        }
      }
    ],
    "notes": [
      // ... les notes
    ],
    "parametrage": {
      // ... les options en fonction des établissements
    }
  }
}

En gros dans la liste des périodes (qui comprend donc la liste des matières) il y a les codes A00X qui semblent être ceux des trimestres/semestres qui sont ce qui nous intéresse vu que les notes sont reliées à ces périodes. Mais il y a aussi d'autres "périodes", dans l'exemple que j'ai j'ai un relevé et un truc annuel, de mon côté ED n'affiche pas ça donc je ne sais pas du tout à quoi ça correspond mais peut-être qu'il y a un problème lors du parsing à cause de ça parce que le nombre de période, leur rôle et fonctionnement semble différer d'un établissement et donc d'un utilisateur à l'autre.

Btw j'ai enlevé bcp de clés et de valeurs pour que ce soit plus simple à comprendre mais la version full est dispo ici (un peu de pub pour edp au passage xD)

imyanice · 2024-10-28T09:30:38Z

@imyanice la gestion des notes de ED est un peu bizarre (de toutes leurs API de ED il me semble que c'est vraiment la pire à parse)

La réponse ressemble en gros à ça :

{

  "code": 200,

  "token": "token",

  "host": "HTTP46",

  "data": {

    "periodes": [

      {

        "idPeriode": "A001",

        "codePeriode": "A001",

        "periode": "1er Trimestre",

        "annuel": false,

        "examenBlanc": false,

        "cloture": true,

        "ensembleMatieres": {

          "disciplines": [

            // une liste des matières de ce trimestre/semestre

            ]

        }

      },

      {

        "idPeriode": "A001R001",

        "codePeriode": "A001R001",

        "periode": "Relev\u00e9 ",

        "ensembleMatieres": {

          "disciplines": [

            // liste des matière ...

          ]

        }

      },

      {

        "idPeriode": "A002",

        "codePeriode": "A002",

        "periode": "2\u00e8me Trimestre",

        "ensembleMatieres": {

          "disciplines": [

            // ...

          ]

        }

      },

      {

        "idPeriode": "A003",

        "codePeriode": "A003",

        "periode": "3\u00e8me Trimestre",

        "ensembleMatieres": {

          "disciplines": [

            // ...

          ]

        }

      },

      {

        "idPeriode": "A999Z",

        "codePeriode": "A999Z",

        "periode": "Ann\u00e9e",

        "ensembleMatieres": {

          "disciplines": [

            // ...

          ]

        }

      }

    ],

    "notes": [

      // ... les notes

    ],

    "parametrage": {

      // ... les options en fonction des établissements

    }

  }

}

En gros dans la liste des périodes (qui comprend donc la liste des matières) il y a les codes A00X qui semblent être ceux des trimestres/semestres qui sont ce qui nous intéresse vu que les notes sont reliées à ces périodes. Mais il y a aussi d'autres "périodes", dans l'exemple que j'ai j'ai un relevé et un truc annuel, de mon côté ED n'affiche pas ça donc je ne sais pas du tout à quoi ça correspond mais peut-être qu'il y a un problème lors du parsing à cause de ça parce que le nombre de période, leur rôle et fonctionnement semble différer d'un établissement et donc d'un utilisateur à l'autre.

Btw j'ai enlevé bcp de clés et de valeurs pour que ce soit plus simple à comprendre mais la version full est dispo ici (un peu de pub pour edp au passage xD)

Nan mais ça je sais, les notes sont déjà implémentés et fonctionnent mais pas pour tout le monde, c'est ça le problème...
Ta période annuel est retournée par le serveur mais non affiché du aux parametrages de ton établissement

saumon-brule · 2024-10-28T10:12:54Z

mais pas pour tout le monde

Justement c'est peut-être l'origine du problème vu que le contenu des réponses d'ED est différent d'un établissement à l'autre. Et vous sembliez partir sur l'hypothèse des périodes mais je sais pas si vous parliez de ça ou pas en tout cas ça m'a fait penser à ces trucs qui pourraient facilement poser problème si le decoder que vous utilisez ne gère pas bien les différentes "périodes" de ED possibles

saumon-brule · 2024-10-28T10:14:23Z

Mais encore une fois je suis pas sûr de savoir de quoi vous parlez avec les périodes je drop juste l'info ici au cas où. Il vaut mieux prévenir que guérir

imyanice · 2024-10-28T11:20:38Z

d'acc, merci!

imyanice · 2024-10-28T13:05:27Z

resolved with LiterateInk/Pawdirecte#7

Kgeek33 mentioned this issue Oct 28, 2024

[BUG] Quasiment aucune infos ne s'affichent pour ED #316

Closed

imyanice mentioned this issue Oct 28, 2024

fix: erroneous conditional logic LiterateInk/Pawdirecte#7

Merged

tryon-dev closed this as completed in cd4db50 Oct 28, 2024

Kgeek33 mentioned this issue Nov 16, 2024

fix(home) : Android en version 7.2.0 et iOS en version 7.3.0 de la page d'accueil #372

Open

9 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Bug]: Notes qui chargent indéfiniment #301

[Bug]: Notes qui chargent indéfiniment #301

okayhappex commented Oct 22, 2024

Kgeek33 commented Oct 22, 2024

okayhappex commented Oct 22, 2024

Kgeek33 commented Oct 22, 2024

okayhappex commented Oct 22, 2024

Kgeek33 commented Oct 22, 2024

Kgeek33 commented Oct 22, 2024

okayhappex commented Oct 22, 2024

Evorsiooo commented Oct 23, 2024

Gabriel29306 commented Oct 23, 2024

Kgeek33 commented Oct 23, 2024

Kgeek33 commented Oct 23, 2024

Kgeek33 commented Oct 23, 2024

okayhappex commented Oct 24, 2024

imyanice commented Oct 24, 2024

saumon-brule commented Oct 27, 2024 •

edited

Loading

Evorsiooo commented Oct 27, 2024

imyanice commented Oct 28, 2024

saumon-brule commented Oct 28, 2024

imyanice commented Oct 28, 2024

saumon-brule commented Oct 28, 2024 •

edited

Loading

saumon-brule commented Oct 28, 2024

imyanice commented Oct 28, 2024

imyanice commented Oct 28, 2024 •

edited

Loading

[Bug]: Notes qui chargent indéfiniment #301

[Bug]: Notes qui chargent indéfiniment #301

Comments

okayhappex commented Oct 22, 2024

Description du bug

Étapes à reproduire

Comportement attendu

Appareil

Version du système d`exploitation

Version

Environnement

Source de l'application

Service scolaire

Captures d'écran / vidéo

Contexte supplémentaire

Kgeek33 commented Oct 22, 2024

okayhappex commented Oct 22, 2024

Kgeek33 commented Oct 22, 2024

okayhappex commented Oct 22, 2024

Kgeek33 commented Oct 22, 2024

Kgeek33 commented Oct 22, 2024

okayhappex commented Oct 22, 2024

Evorsiooo commented Oct 23, 2024

Gabriel29306 commented Oct 23, 2024

Kgeek33 commented Oct 23, 2024

Kgeek33 commented Oct 23, 2024

Kgeek33 commented Oct 23, 2024

okayhappex commented Oct 24, 2024

imyanice commented Oct 24, 2024

saumon-brule commented Oct 27, 2024 • edited Loading

Evorsiooo commented Oct 27, 2024

imyanice commented Oct 28, 2024

saumon-brule commented Oct 28, 2024

imyanice commented Oct 28, 2024

saumon-brule commented Oct 28, 2024 • edited Loading

saumon-brule commented Oct 28, 2024

imyanice commented Oct 28, 2024

imyanice commented Oct 28, 2024 • edited Loading

saumon-brule commented Oct 27, 2024 •

edited

Loading

saumon-brule commented Oct 28, 2024 •

edited

Loading

imyanice commented Oct 28, 2024 •

edited

Loading