Skip to content

Fix: Réparation du service AppScho#694

Open
Orafilynie wants to merge 2 commits intoPapillonApp:mainfrom
Orafilynie:fix-appscho
Open

Fix: Réparation du service AppScho#694
Orafilynie wants to merge 2 commits intoPapillonApp:mainfrom
Orafilynie:fix-appscho

Conversation

@Orafilynie
Copy link
Contributor

Contribution

Règles de contribution

Caution

Afin de garantir une application stable et pérenne dans le temps, nous t'invitons à vérifier que tu as bien respecté les règles de contribution. Sans cela, ta Pull Request ne pourra pas être examinée.

  • Cette Pull Request porte sur une seule fonctionnalité ou un seul correctif.
  • Cette Pull Request n'est pas faite essentiellement avec de l'IA.
  • Pour tout changement majeur, j’ai créé une issue afin d’échanger avec les mainteneurs de Papillon sur la meilleure façon de l’intégrer.
  • Ma Pull Request respecte les conventions Conventional Commits et Conventional Branch ainsi que les conventions de codage de l'application.
  • J’ai testé mes modifications sur iOS et Android, et l’application fonctionne correctement.
  • J’emploie un langage informel, clair et concis dans mes messages.
  • J’ai documenté mes changements de manière appropriée, soit dans la description de la Pull Request, soit dans le GitBook.
  • J’ai ajouté les traductions nécessaires dans au moins un fichier de langue.

Résumé des changements

Utilisation d'un refresh manuel via la lecture du state des réponses API pour éviter des refresh frénétiques du service pouvant causer un token desync pour le refresh, entrainent une perte permanenta du token.

Réparation de l’emploi n'affichant pas les cours du a un fuseau horaire hard codé.

Instance UPJV
Une future PR sera ouverte pour la mise a jour de la dépendance AppScho.js une fois les réparations menées dans celle-ci approuvées pour le fonctionnement de cette instance.

Capture(s) d'écran

Note

Si tes changements concernent l'interface utilisateur, inclue des captures d'écran pour illustrer tes modifications.

Informations supplémentaires

Note

Numéro des issues concernées par cette Pull Request, détail sur le fonctionnement ou les choix techniques effectués, ainsi que toute autre information pertinente.

Copy link
Contributor

@Dev-LeChacal Dev-LeChacal left a comment

Choose a reason for hiding this comment

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

LGTM

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

Successfully merging this pull request may close these issues.

2 participants

Comments