-
Notifications
You must be signed in to change notification settings - Fork 0
WIP feat: Onboarding dropy #211
base: main
Are you sure you want to change the base?
Conversation
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 fais juste une review partielle.
- Est-ce qu'on peut renommer welcomeDropy en onboardingDropy ?
- Pour la photo du drop on va en discuter en réunion
- Faudra probablement étendre le système pour que le dev envoit automatiquement un message dès que le drop et sa conv sont ouverts.
const randLatitude = latitude + (Math.random() - 0.2) * 0.01; | ||
const randLongitude = longitude + (Math.random() - 0.2) * 0.01; |
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.
Met un offset fixe, pas besoin de forcement randomiser et ca evitera d'avoir un drop qui se pose exactement sur la position de l'utilisateur
J'ai aussi pensé au fait que seul l'utilisateur qui vient de créer son compte doit pouvoir récupérer ce drop et personne d'autre. Pour le message auto quand la conv est ouverte ça va être full front ? Genre c'est juste un on message du socket mais le message c'est le message auto envoyé par le front. Je sais pas si c'est faisable actuellement mais surement pas besoin d'une route dédiée pour ça. |
On peut ajouter un champ
|
Donc dans le socket de creation de conv, on check si ya le onboardingUser, si oui on lui envoie un message avec le compte du dev |
POST : /dropy/welcomeDropy (latitude, longitude)
Fait apparaître un drop proche des coordonnées données, sélectionne un compte dev random comme emitter, prend sa photo de profil comme image de drop (pour le moment).
Pas pu tester si le drop apparait assez proche car j'arrive pas a build sur mon front je sais pas pourquoi.