Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

WIP feat: Onboarding dropy #211

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

WIP feat: Onboarding dropy #211

wants to merge 3 commits into from

Conversation

tjobit
Copy link
Contributor

@tjobit tjobit commented Dec 30, 2022

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.

@tjobit tjobit requested a review from celian-rib December 30, 2022 18:05
@tjobit tjobit self-assigned this Dec 30, 2022
@tjobit tjobit linked an issue Dec 30, 2022 that may be closed by this pull request
Copy link
Contributor

@celian-rib celian-rib left a 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.

Comment on lines 173 to 174
const randLatitude = latitude + (Math.random() - 0.2) * 0.01;
const randLongitude = longitude + (Math.random() - 0.2) * 0.01;
Copy link
Contributor

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

@tjobit
Copy link
Contributor Author

tjobit commented Jan 1, 2023

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.

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.

@celian-rib
Copy link
Contributor

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.

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 onboardingUser sur le drop.
Si ce champ n'est pas null alors :

  • Seul le onboardingUser verra le drop.
  • Des que le chat est ouvert avec ce drop, le message d'accueil est automatiquement envoyé par le emitter (Le compte dev).

Tout ca c'est que du back

@tjobit
Copy link
Contributor Author

tjobit commented Jan 1, 2023

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.

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 onboardingUser sur le drop. Si ce champ n'est pas null alors :

  • Seul le onboardingUser verra le drop.
  • Des que le chat est ouvert avec ce drop, le message d'accueil est automatiquement envoyé par le emitter (Le compte dev).

Tout ca c'est que du back

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

@celian-rib celian-rib changed the title feat: add a welcome drop near user feat: Onboarding dropy Jan 1, 2023
@celian-rib celian-rib added feature New feature or request socket labels Jan 1, 2023
@tjobit tjobit changed the title feat: Onboarding dropy WIP feat: Onboarding dropy Jan 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature New feature or request socket
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Welcome Drop
2 participants