Skip to content

Brouillon: fix(@multi-frontend) problèmes back button Android#89

Draft
benjhoo wants to merge 4 commits intodevelopfrom
fix/MULTI-6193-back-button-android
Draft

Brouillon: fix(@multi-frontend) problèmes back button Android#89
benjhoo wants to merge 4 commits intodevelopfrom
fix/MULTI-6193-back-button-android

Conversation

@benjhoo
Copy link
Collaborator

@benjhoo benjhoo commented Nov 7, 2024

Checklist de PR

Veuillez vérifier que votre PR respecte bien les indications suivantes :

  • Votre PR pointe vers la branche develop
  • Votre PR suit les différentes étapes du guide de contribution : https://www.esup-portail.org/wiki/x/KQCeUQ
  • Les modifications ont été testées de votre côté, cela implique également des tests sur des périphériques (iOS + Android) si le client a évolué
  • La documentation a été mise à jour et prend en compte les changements (fichiers README, Wiki Esup)

Type de PR

Quel type de changement concerne cette PR ?

  • Bug Fix
  • Nouvelle Feature
  • Mise à jour de la documentation (README, CHANGELOG, CONTRIBUTING)
  • Style (SCSS, Assets)
  • Refactoring de code
  • Ajout de tests
  • Build (scripts npm, .sh)
  • CI
  • Chore (nouvelle Release, maj de dépendances)
  • Revert

Quel est le comportement actuel ?

Le bouton back sur les téléphones Android ne permet pas de revenir à l'accueil du téléphone lorsque l'utilisateur se trouve sur la page d'accueil de l'application.
Bug 6193 (gestion en interne)

Quel est le nouveau comportement ?

Comportement attendu, on minimise l'app.

Cette PR implique un Breaking Change ?

  • Oui
  • Non

Information complémentaire

RAS

@benjhoo
Copy link
Collaborator Author

benjhoo commented Nov 7, 2024

Ne pas merger pour le moment car il semble y avoir des erreurs au niveau de la gestion de l'historique qui fait que le retour arrière n'est pas correct.
J'aimerai que ce problème soit corrigé avec cette PR

@ActxLeToucan ActxLeToucan marked this pull request as draft November 18, 2024 14:41
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.

1 participant