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

Améliorer les suggestions [NGC-1191] [NGC-1217] #694

Open
wants to merge 18 commits into
base: preprod
Choose a base branch
from

Conversation

florianpanchout
Copy link
Contributor

No description provided.

Copy link

Copy link

vercel bot commented Sep 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nosgestesclimat ❌ Failed (Inspect) Sep 19, 2024 9:44am

Copy link

github-actions bot commented Sep 10, 2024

Report for the pull request #694


🌐 Translation status

UI's texts

Language Nb. missing translations Status
en Ø ✔️
es Ø ✔️

FAQ's questions

Language Nb. missing translations Status
en Ø ✔️
es Ø ✔️

You will find more information about the translation in the dedicated file.

Copy link

@florianpanchout florianpanchout marked this pull request as ready for review September 19, 2024 06:50
@florianpanchout florianpanchout changed the title Améliorer les suggestions [NGC-1191] Améliorer les suggestions [NGC-1191] [NGC-1217] Sep 19, 2024

const handleSuggestionClick = useCallback(
(suggestion: FormattedSuggestion) => {
trackEvent(
Copy link
Contributor

Choose a reason for hiding this comment

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

Est-ce qu'il ne faudrait pas modifier ce tracker pour faire en sorte d'y inclure le nombre de fois que la suggestion aura été sélectionnée ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pas sur que ça ait un interêt. L'idée de ce tracking c'est de savoir quelles suggestions sont cliquées. Pour avoir les valeurs moyennes on a d'autres trackers

src/components/form/question/Suggestions.tsx Outdated Show resolved Hide resolved
src/components/form/question/NumberInput.tsx Show resolved Hide resolved
src/components/form/question/Suggestions.tsx Outdated Show resolved Hide resolved
@florianpanchout
Copy link
Contributor Author

Mis en pause en attendant les UI

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

Successfully merging this pull request may close these issues.

2 participants