PapillonApp / Papillon

Une alternative open source aux applis de vie scolaire. L'allié de tous les étudiants. Le futur de l'éducation numérique libre et ouverte.
https://papillon.bzh
GNU General Public License v3.0
75 stars 27 forks source link

[Bug]: EDT, impossible de retourner sur un jour passé. #162

Closed Margouta closed 2 weeks ago

Margouta commented 2 weeks ago

Description du bug

Admettons nous sommes le 5 juillet après une journée de cours. Impossible de cliquer sur le 4 (via le sélecteurs calendrier ou boutons sur les cotés). Par défaut l'appli affiche la journée du 5, tant qu'on est le 5. Si je switch sur le 6, impossible de retourner sur la page du 5 (nécessaire de restart l'appli)

Étapes à reproduire

  1. Ouvrir Papillon sur l'onglet agenda.
  2. Changer pour une date antérieure à celle du jour.

Comportement attendu

Afficher l'EDT

Appareil

Samsung Galaxy A33 5G

Version du système d`exploitation

Android 14

Version

7.0.1-beta

Environnement

Application native Android

Source de l'application

Github

Service scolaire

🦋 Pronote

Captures d'écran / vidéo

No response

Contexte supplémentaire

No response

Louis-htmlcss commented 2 weeks ago

Yes pareil pour moi

MAG3845 commented 2 weeks ago

Je confirme avec la dernières version téléchargée depuis Github, fonctionnait niquel en 7.0.

Device : Poco X6 Pro Android 14 HyperOS

Crazzynel commented 2 weeks ago

Yes pareil aussi. Mais je crois avoir trouvé un moyen de corriger ça. Du moins je regarde si il y a un truc a faire...

Appareil: Redmi Note 13 Pro 5G Android 14 HyperOS

Crazzynel commented 2 weeks ago

@Margouta

Je pense avoir trouvé le problème... je te tiens au courant !

MAG3845 commented 2 weeks ago

Merci ! J'ai pas les compétences pour check le code source même si je pense que c'est un tout petit bug ! Ping moi quand t'aura patch et fait ton PR, si j'ai le temps j'essayerai de build l'app et de l'install ! plus sysadmin moi de base, pas Dev

Crazzynel commented 2 weeks ago

@MAG3845 je vois x))) Y a pas de souci. Dans mon lycée on fait ça donc j'en profite... Jsp si c'est possible mais j'aimerais bien pouvoir être assigné à ce bug. Histoire qu'il n'y ait pas de malentendus haha

et vu que perso jsuis plus dev que sysadmin x))

MAG3845 commented 2 weeks ago

Au pire ouvre une PR juste rapidement, et tu fous un link du bug

Crazzynel commented 2 weeks ago

Je préfère le faire proprement mdrrr.

Rexxt commented 2 weeks ago

si tu veux être assigné normalement faut faire partie de l'orga, mais du coup faire proprement c'est avec une PR

Crazzynel commented 2 weeks ago

euhhh

@MAG3845 @Margouta je crois que je l'ai patch involontairement x)))

J'ai lancé un build de l'app sur ma fork. Je te tiens au courant mais je crois que je l'ai patch en modifiant le widget de la page d'accueil... du moins le bug n'a pas l'air de se produire ;-;

Je te tiens au courant.

Crazzynel commented 2 weeks ago

Okkkk... @MAG3845 @Margouta J'ai réussi à résoudrele problème. Il manquait une ligne... j'ai fini mtn... jsuis bon mtn x) Il faudrait ptetre que tu ailles sur ma fork de papillon, que tu testes avec l'apk que j'ai généré sur ma fork tout à l'heure... (Jsp si j'ai le droit)

Car je sais pas quoi commit, vu que je ne retouché qu'une seule ligne de code mdrr... donc bon...

Louis-htmlcss commented 2 weeks ago

Okkkk... @MAG3845 @Margouta J'ai réussi à résoudrele problème. Il manquait une ligne... j'ai fini mtn... jsuis bon mtn x) Il faudrait ptetre que tu ailles sur ma fork de papillon, que tu testes avec l'apk que j'ai généré sur ma fork tout à l'heure... (Jsp si j'ai le droit)

Car je sais pas quoi commit, vu que je ne retouché qu'une seule ligne de code mdrr... donc bon...

Fais ta pr, pour moi ça marche

Crazzynel commented 2 weeks ago

@Louis-htmlcss

Justement... j'ai peur que ça vienne de la modification que j'ai faite lors de la modification du widget notes. Je n'ai absolument aucune modif de l'instance emploi du temps... je sais plus ce que j'ai modifié...

Crazzynel commented 2 weeks ago

Donc je ne peux pas faire de pr...

Louis-htmlcss commented 2 weeks ago

@Louis-htmlcss

Justement... j'ai peur que ça vienne de la modification que j'ai faite lors de la modification du widget notes. Je n'ai absolument aucune modif de l'instance emploi du temps... je sais plus ce que j'ai modifié...

Sinon checkout ta pr pour modif les widgets et regarde si ça fix le pb

ecnivtwelve commented 2 weeks ago

Normalement réparé avec la réécriture du calendrier : https://github.com/PapillonApp/Papillon/pull/181