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

fonctionnalité pouces haut et bas à intégrer #214

Closed
jrduscher opened this issue Oct 22, 2018 · 10 comments
Closed

fonctionnalité pouces haut et bas à intégrer #214

jrduscher opened this issue Oct 22, 2018 · 10 comments
Assignees
Milestone

Comments

@jrduscher
Copy link
Contributor

https://trello.com/c/8buyKsyc/280-int%C3%A9grer-les-fonctionnalit%C3%A9s-des-indicateurs-thumbsup-thumbsdown

@jrduscher jrduscher added this to the 1.4 milestone Oct 22, 2018
@jeremiecook
Copy link
Contributor

popin de feedback

@lionelB
Copy link
Contributor

lionelB commented Nov 6, 2018

@jeremiecook la popin est censée arriver une fois le pouce bas clické ?

@jeremiecook
Copy link
Contributor

Tout à fait ! Ou bien en cliquant sur les blocs "Vous n'avez pas trouvé votre réponse ?" sous les listes de réponses

@lionelB
Copy link
Contributor

lionelB commented Nov 6, 2018

@jeremiecook merci !

@lionelB
Copy link
Contributor

lionelB commented Nov 9, 2018

@jeremiecook @jrduscher Je me pose la question des liens "les réponses qui pourraient vous aider"
Ça me parait étrange de lui représenter les 3 premiers resultats qu'on lui a présenté dans le suggester quand il a tapé sa question, vu que ça ne l'aide pas à réalisé son objectif. Est que j'ai raté un trucs ?

@jeremiecook
Copy link
Contributor

@lionelB Hello ! Je pense que ces résultats ne s'affichent que si l'utilisateur complète, modifie ou affine le champ "question" de cette popin. C'est un moyen de valider que la réponse n'existe pas déjà avant d'envoyer son retour. L'utilisateur n'a pas toujours saisi de question avant d'arriver sur cette page (par exemple, s'il a cliqué sur un thème / dossier puis consulté la liste des réponses)

@jeremiecook
Copy link
Contributor

@lionelB Cela dit, je pense que ce bloc peut rester masqué à l'ouverture de la popin

@lionelB
Copy link
Contributor

lionelB commented Nov 9, 2018

@jrduscher la PR #264 va répondre partiellement à cette issue car elle ne gére que le pousse bas.
pour le pousse haut, cela va demander un autre travail plus conséquent (impact sur ES / api)
Je propose qu'une fois la PR mergée, on ferme cette issue et on ouvre une pour le pouce haut ?

@lionelB
Copy link
Contributor

lionelB commented Nov 9, 2018

ca sera l'occasion de fermer l'issue #134

@lionelB
Copy link
Contributor

lionelB commented Nov 19, 2018

closed by #264

pouce haut #294

@lionelB lionelB closed this as completed Nov 19, 2018
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

No branches or pull requests

4 participants