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

[FEATURE] Mise à jour du CTA de fin de parcours autonomes (PIX-15070). #10491

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

Jeyffrey
Copy link
Contributor

@Jeyffrey Jeyffrey commented Nov 5, 2024

🍂 Problème

On ne veut pas renvoyer vers le dashboard à la fin d'un parcours autonome avec un utilisateur non connecté.

🌰 Proposition

Le CTA indique maintenant qu'il dirige vers l'écran de connexion.

🎃 Remarques

Il a fallu revoir la redirection dans la route du login car elle laissait passer les utilisateurs anonymes (considérés comme connectés, avec un token).

🪵 Pour tester

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

@@ -164,7 +164,7 @@ export default class EvaluationResultsHero extends Component {
{{else}}
{{#if this.isAutonomousCourse}}
<PixButtonLink @route="authentication.login" @size="large">
{{t "navigation.back-to-homepage"}}
{{if this.currentUser.user.isAnonymous (t "common.actions.login") (t "navigation.back-to-homepage")}}
Copy link
Contributor

Choose a reason for hiding this comment

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

question
Est-ce qu'il ne faut pas flush le service avec le user isAnonymous ?
le isAnonymous c'est seulement le temps du parcrous. si il est arrivé à la fin de son parcours. on devrait "flush" ces données non ? Ou il y a une volonté de la raccrocher avec ces ke le cas échéant si il créer un compte ?

Copy link
Contributor Author

@Jeyffrey Jeyffrey Nov 5, 2024

Choose a reason for hiding this comment

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

On y a pensé mais en fait on s'est dit que l'utilisateur pouvait avoir envie de revenir en arrière, sur son navigateur, après avoir cliqué sur le CTA.
Et si on flush, bah il peut plus voir son résultat

Copy link
Contributor

Choose a reason for hiding this comment

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

C'est que la condition là, va fonctionner pour toutes les campagnes avec des utilisateurs anonymous. pas seulement le parcours autonome . d'où mon interrogation 😅 . en soit ça me choque pas. c'est plus pour être raccord partout.

Copy link
Contributor Author

@Jeyffrey Jeyffrey Nov 6, 2024

Choose a reason for hiding this comment

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

Pour l'instant c'est vraiment un pansement.
Y a la @1024pix/team-acces qui devrait se pencher sur le sujet des utilisateurs anonymes et de la navigation dans l'app.

@AlexNaudyy
Copy link
Contributor

Le loader apparait en review app^^ Allez on l'enlève aussi ici!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants