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

[TECH] Suppression de l'utilisation de la dependency RSVP partout où on peut immédiatement la remplacer par Promise (PIX-15814) #10841

Merged
merged 2 commits into from
Dec 20, 2024

Conversation

lego-technix
Copy link
Contributor

@lego-technix lego-technix commented Dec 17, 2024

🎄 Problème

Le code de Pix des applications Front utilise la dependency obsolète RSVP pour des usages qui peuvent être immédiatement remplacés par l'utilisation des Promises natives. On cherche à se débarrasser de la [dependency obsolète RSVP.

🎁 Proposition

Remplacer l'utilisation de RSVP par l'utilisation de Promises natives partout où le remplacement est immédiat.

On s'occupera de remplacer RSVP.hash dans une autre PR.

🧦 Remarques

RAS

🎅 Pour tester

Tests de non-régression sur la RA

  • Se connecter avec un identifiant/email et un mot de passe à Pix App et constater qu'il n'y a pas de régression
  • Se déconnecter de Pix App et constater qu'il n'y a pas de régression

Tests de non-régression en local

Sur Pix App avec un SSO OIDC :

  • Se connecter à Pix App avec un SSO OIDC et constater qu'il n'y a pas de régression
  • Se déconnecter de Pix App et constater qu'il n'y a pas de régression

Sur Pix App avec le Gar :

  • Se connecter à Pix App avec le Gar et constater qu'il n'y a pas de régression
  • Se déconnecter de Pix App et constater qu'il n'y a pas de régression

Sur Pix Admin avec un SSO OIDC :

  • Se connecter à Pix Admin avec un SSO OIDC et constater qu'il n'y a pas de régression
  • Se déconnecter de Pix Admin et constater qu'il n'y a pas de régression

@lego-technix lego-technix self-assigned this Dec 17, 2024
@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 :

@lego-technix lego-technix changed the title Refactor remove rsvp obsolete dependency [TECH] Suppression de l'utilisation de la dependency RSVP partout où on peut immédiatement la remplacer par Promise Dec 17, 2024
@lego-technix lego-technix marked this pull request as ready for review December 17, 2024 17:18
@lego-technix lego-technix requested a review from a team as a code owner December 17, 2024 17:18
@lego-technix lego-technix changed the title [TECH] Suppression de l'utilisation de la dependency RSVP partout où on peut immédiatement la remplacer par Promise [TECH] Suppression de l'utilisation de la dependency RSVP partout où on peut immédiatement la remplacer par Promise (PIX-15814) Dec 19, 2024
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

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

✅ Revue et testée avec succès 🎉

Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay left a comment

Choose a reason for hiding this comment

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

Lu et testé avec succès

@pix-service-auto-merge pix-service-auto-merge force-pushed the refactor-remove-rsvp-obsolete-dependency branch from ed1d756 to 4404721 Compare December 20, 2024 17:57
@pix-service-auto-merge pix-service-auto-merge merged commit d4f81a4 into dev Dec 20, 2024
8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the refactor-remove-rsvp-obsolete-dependency branch December 20, 2024 18:05
HEYGUL pushed a commit that referenced this pull request Dec 24, 2024
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