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

[BUGFIX] Créer une session de certification sur d'anciens navigateurs (PIX-11269) #8200

Merged

Conversation

Osirisxxl
Copy link
Contributor

@Osirisxxl Osirisxxl commented Feb 27, 2024

🦄 Problème

Lorsque l’utilisateur utilise son navigateur Firefox 91.0 et essaie de cliquer sur le bouton “Créer une session”, rien ne se passe. La création en masse de session quant à elle fonctionne (bouton d'à côté “Créer/éditer plusieurs sessions”)

La méthode crypto.randomUUID n'existe pas sur cette version de navigateur.

🤖 Proposition

Mettre en place le polyfill de cette méthode proposé par ember-data.

🌈 Remarques

La doc de polyfill ne le mentionne pas, mais depuis la version 4.12 de ember-data, le simple ajout du polyfill ne fonctionne pas. Il faut également que embroider/macros soit installé et listé dans les dépendances du package.json pour que le polyfill soit installé. cf cette issue.

Je propose également le listing de la version sur mon-pix même s'il n'y a pas de bug actuellement. Concrètement, c'est à l'utilisation de la config pour mettre en place le polyfill que la dépendance aurait dû être listée. Ainsi, on évitera le même bug silencieux lors de la montée d'ember-data de mon-pix en 4.12.

PI: un tableau montrant la disponibilité de randomUUID sur chaque navigateur

💯 Pour tester

Installer une ancienne version de firefox e.g la 91.
Se connecter sur pix certif (certif-pro@example.net/pix123)
Créer une nouvelle session

Non régression:
Mêmes étapes sur un chrome à jour par exemple.

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@AndreiaPena
Copy link
Member

Testé func avec Lambda Test en firefox 91
Capture d’écran 2024-02-29 à 14 45 19

@@ -22,6 +22,13 @@ module.exports = function (defaults) {
'ember-cli-template-lint': {
testGenerator: 'qunit',
},
'@embroider/macros': {
Copy link
Member

Choose a reason for hiding this comment

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

j'allais poser la question de pourquoi je ne trouve pas celui de pix App mais il existait déjà dans le code, donc ✅

…polyfill for randomUUID

Signed-off-by: Jérémie Picard <jeremie.picard@pix.fr>
Signed-off-by: Jérémie Picard <jeremie.picard@pix.fr>
@pix-service-auto-merge pix-service-auto-merge force-pushed the PIX-11269-polyfill-randomuuid-on-old-navigators branch from 7d5c939 to 8109e63 Compare February 29, 2024 15:55
@pix-service-auto-merge pix-service-auto-merge merged commit 0885714 into dev Feb 29, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-11269-polyfill-randomuuid-on-old-navigators branch February 29, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants