-
Notifications
You must be signed in to change notification settings - Fork 40
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
(PC-33573)[PRO] feat: add wonderpush to project #15526
base: master
Are you sure you want to change the base?
Conversation
ffb9fa1
to
c7e778e
Compare
Visit the preview URL for this PR (updated for commit 137a126): https://pc-pro-testing--pr15526-tcoudray-pass-pc-335-dguc422p.web.app (expires Fri, 24 Jan 2025 08:31:15 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1 |
@@ -69,6 +69,7 @@ | |||
"react-instantsearch": "^7.13.8", | |||
"react-redux": "^9.2.0", | |||
"react-router-dom": "^6.28.0", | |||
"react-wonderpush": "^1.0.9", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je me permets de challenger cette lib, est-ce qu'on a besoin d'une nouvelle librairie de notifications sachant qu'on a déjà hotjar qui peut faire ça (je crois) ?
Je vois que cette surcouche react est pas méga utilisée (97 téléchargement cette semaine, 1 étoile sur github), elle n'a pas eu de màj depuis avril 2023 et elle n'a pas de lib de types 🤔
Et dernier point, il faudra voir si cette lib crée des notifs conformes au RGAA, ce qui est pas évident à faire! (par exemple, est-ce que les notifs sont annoncées aux lecteurs d'écran à leur apparition). Pour ça est-ce qu'il y a déjà un moyen pour tester en local ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
L'idée est qu'elle remplace hotjar à terme, c'est ce que souhaite l'équipe marketing pour pouvoir tout gérer depuis Brevo.
Pour le point 2, je suis d'accord avec toi. C'est cette lib que recommande Brevo dans sa documentation.
Pour le point 3, je ne saurais pas te répondre. Je ne crois pas qu'il soit possible de tester en local. L'idée était de faire un test en testing avec Nine.
pro/src/Root.tsx
Outdated
<AppRouter /> | ||
</StoreProvider> | ||
</Provider> | ||
<WonderPush options={{webKey: WONDER_PUSH_WEB_KEY}}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il faudrait demander le consentement à l'utilisateur pour l'utilisation des données (https://docs.wonderpush.com/docs/gdpr-compliance). On fait ça d'habitude dans app/analytics
via le orejimeConfig.ts
c7e778e
to
98cd3ca
Compare
a7ea23f
to
06e4002
Compare
06e4002
to
b3bbc76
Compare
b3bbc76
to
137a126
Compare
But de la pull request
Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-33573
Librairie de notification web de Brevo (documentation).
Vérifications