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

Gestion des événements remontés par RDV-I via un webhook [GEN-1765] #4563

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

leo-naeka
Copy link
Contributor

🤔 Pourquoi ?

Pour gérer les événements remontés par RDV-I et tout synchroniser

🏝️ Comment tester

  • S'inscrire en tant que candidat avec un email valide

  • Puis postuler auprès d'un SIAE

  • En tant qu'employeur de ce SIAE, s'assurer que l'ID d'organisation RDV-I est bien défini en admin (654 pour la démo)

  • Puis parcourir la liste des candidatures et inviter le candidat à prendre RDV

  • En tant que candidat, je reçois un mail d'invitation de la part de RDV-I

  • Ouvrir le mail et cliquer sur le bouton pour prendre RDV

    • En tant qu'employeur du SIAE, je constate que mon invitation a été ouverte
  • En tant que candidat, choisir un créneau et confirmer le RDV

    • En tant qu'employeur du SIAE, je constate que RDV est à présent listé

Bonus : mises à jour

  • En tant que candidat, modifier le RDV en choisissant et confirmant une autre date

    • En tant qu'employeur du SIAE, je constate que RDV a bien été replanifié à cette autre date
  • En tant qu'agent RDV-I, annuler le RDV (excusé ou à l'initiative du service)

    • En tant qu'employeur du SIAE, je constate que RDV a bien été annulé, avec le bon label

@leo-naeka leo-naeka added the ajouté Ajouté dans le changelog. label Aug 15, 2024
@leo-naeka leo-naeka self-assigned this Aug 15, 2024
@leo-naeka leo-naeka changed the base branch from master to leo/rdvi-job-application-detail August 15, 2024 21:02
@leo-naeka leo-naeka force-pushed the leo/rdvi-job-application-detail branch from f313cb1 to 4f2eb36 Compare August 15, 2024 21:15
@leo-naeka leo-naeka added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Aug 16, 2024
Copy link

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@leo-naeka leo-naeka force-pushed the leo/rdvi-job-application-detail branch 2 times, most recently from b271cb3 to 8f42016 Compare September 10, 2024 07:30
@leo-naeka leo-naeka force-pushed the leo/rdvi-webhooks branch 3 times, most recently from 08c37af to a5aaf0f Compare September 11, 2024 19:22
@leo-naeka leo-naeka force-pushed the leo/rdvi-job-application-detail branch from 8f42016 to a7eae95 Compare September 11, 2024 21:27
@leo-naeka leo-naeka force-pushed the leo/rdvi-webhooks branch 2 times, most recently from 0431f09 to d2e8cd8 Compare September 11, 2024 21:57
@leo-naeka leo-naeka force-pushed the leo/rdvi-job-application-detail branch from a7eae95 to 0c6985f Compare September 11, 2024 21:57
@leo-naeka leo-naeka marked this pull request as ready for review September 11, 2024 22:04
itou/rdv_insertion/migrations/0001_initial.py Show resolved Hide resolved
itou/www/rdv_insertion/urls.py Outdated Show resolved Hide resolved
itou/www/rdv_insertion/views.py Show resolved Hide resolved
itou/www/rdv_insertion/views.py Outdated Show resolved Hide resolved
@francoisfreitag
Copy link
Contributor

francoisfreitag commented Sep 12, 2024

Je ne suis pas du tout au clair sur RDV-I vs RDV-S ? Des champs ont un helptext RDV-I mais verbose_name RDV-S.
Est-ce qu’on ne mettrait pas les noms complets ? Et éventuellement un nommage cohérent ?

Edit: En fait, RDV-I utilise les ID RDV-S pour les rendez-vous.

Copy link
Contributor

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Généralement ça m’a l’air bien, je n’ai pas vérifié le schéma de RDV-I, ni même testé en local, j’ai fait confiance aux tests. 🙈

itou/www/rdv_insertion/views.py Outdated Show resolved Hide resolved
tests/www/rdv_insertion/tests.py Outdated Show resolved Hide resolved
tests/www/rdv_insertion/tests.py Show resolved Hide resolved
tests/www/rdv_insertion/tests.py Show resolved Hide resolved
tests/www/rdv_insertion/tests.py Outdated Show resolved Hide resolved
@leo-naeka
Copy link
Contributor Author

OK pour mettre les noms complets,

Pour le verbose_name je tâche d'être compact en ayant une pensée pour l'admin et par cohérence j'ai mis pareil dans help_text / error_messages. Mais c'est probablement plus clair tu as raison. Je corrigerai dans les early-PRs.

J'ai réalisé sur le tard (= après mon early-PR qui avait ajouté le champ) que l'API RDV-Insertion mélange les IDs RDV-Insertion et RDV-Solidarités. En l'occurrence l'ID des organisations est celui de RDV-Solidarités.

@leo-naeka leo-naeka force-pushed the leo/rdvi-job-application-detail branch from 0c6985f to 50b5e6e Compare September 13, 2024 12:06
@leo-naeka leo-naeka force-pushed the leo/rdvi-job-application-detail branch from 50b5e6e to bc7a39a Compare September 13, 2024 14:44
@leo-naeka leo-naeka force-pushed the leo/rdvi-job-application-detail branch from bc7a39a to 6f0de12 Compare September 13, 2024 14:56
@leo-naeka leo-naeka removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Sep 13, 2024
@leo-naeka leo-naeka merged commit 6bb1ddb into leo/rdvi-job-application-detail Sep 13, 2024
13 checks passed
@leo-naeka leo-naeka deleted the leo/rdvi-webhooks branch September 13, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ajouté Ajouté dans le changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants