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] Ajouter des paramètres dans l'URL du lien custom de fin de parcours (PIX-15184). #10505

Conversation

Jeyffrey
Copy link
Contributor

@Jeyffrey Jeyffrey commented Nov 6, 2024

🍂 Problème

Lors de la création de la nouvelle page de fin de parcours, l'intégration de search params dans le lien custom de fin de parcours a été oublié.

🌰 Proposition

Permettre l'ajout de 3 paramètres dans l'URL custom de fin de parcours (stage, masteryRate, externalId).

🎃 Remarques

Ce code existait ici :

🪵 Pour tester

  • En RA, se connecter avec [email protected]
  • Voir ce parcours
  • Constater que l'URL du bouton du bloc "Message de votre organisation" a bien des les 3 query params souhaités.

@Jeyffrey Jeyffrey self-assigned this Nov 6, 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 :

@Jeyffrey Jeyffrey force-pushed the pix-15184-custom-organization-results-block-link-should-accept-params branch 2 times, most recently from a4db38a to f4d3f07 Compare November 6, 2024 17:43
@xav-car
Copy link
Contributor

xav-car commented Nov 7, 2024

func ko :
il manque le parseIntau calcul du masteryPercentage

image

xav-car
xav-car previously approved these changes Nov 7, 2024
@xav-car xav-car self-requested a review November 7, 2024 10:26
@xav-car xav-car dismissed their stale review November 7, 2024 10:26

Suite à une seconde func, on a vu qu'il manquait un petit parseInt

@Jeyffrey Jeyffrey force-pushed the pix-15184-custom-organization-results-block-link-should-accept-params branch from f4d3f07 to 5c35847 Compare November 7, 2024 10:57
@Jeyffrey
Copy link
Contributor Author

Jeyffrey commented Nov 7, 2024

func ko : il manque le parseIntau calcul du masteryPercentage

C'est modifié

@xav-car
Copy link
Contributor

xav-car commented Nov 7, 2024

image
OK

@xav-car xav-car changed the title [FEATURE] Ajouter des paramètres dans l'URL du lien custom de fin de parcours (PIX-15184). [BUGFIX] Ajouter des paramètres dans l'URL du lien custom de fin de parcours (PIX-15184). Nov 7, 2024
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15184-custom-organization-results-block-link-should-accept-params branch from 5c35847 to c063a3a Compare November 7, 2024 13:02
@pix-service-auto-merge pix-service-auto-merge merged commit 2164c3f into dev Nov 7, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15184-custom-organization-results-block-link-should-accept-params branch November 7, 2024 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Func Review OK PO validated functionally the PR 🚀 Ready to Merge team-evaluation PR relatives à l'expérience d'évaluation Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants