entrouvert
/
ants-hub
15
0
Fork 0
Code Issues Pull Requests 3 Releases Wiki Activity

api: send a correct management_url to /api/appointments (#89529) #16

Merged
bdauvergne merged 1 commits from wip/89529-L-URL-de-gestion-envoye-dans-l-A into main 2024-04-15 10:55:57 +02:00
Conversation 2 Commits 1 Files Changed 5 +23 -13
bdauvergne commented 2024-04-14 13:06:59 +02:00
Owner
Copy Link
No description provided.
bdauvergne added 1 commit 2024-04-14 13:06:59 +02:00
nroche commented 2024-04-15 10:29:34 +02:00
Owner
Copy Link

J'ai pas tout compris (surtout le lien avec le ticket client).
Est-ce que tu pourrais, vite fait, m'indiquer la différence entre make_gestion_url et get_gestion_url, stp ?

et je me répond moi même :

fix pour l’envoi de la bonne URL de gestion du rdv dans leur base de dedoublonnage, ça bloque actuellement l’annulation des rdvs.

J'ai pas tout compris (surtout le lien avec le ticket client). Est-ce que tu pourrais, vite fait, m'indiquer la différence entre make_gestion_url et get_gestion_url, stp ? et je me répond moi même : > fix pour l’envoi de la bonne URL de gestion du rdv dans leur base de dedoublonnage, ça bloque actuellement l’annulation des rdvs.
nroche approved these changes 2024-04-15 10:37:09 +02:00
bdauvergne commented 2024-04-15 10:55:47 +02:00
Author
Owner
Copy Link

J'ai pas tout compris (surtout le lien avec le ticket client).
Est-ce que tu pourrais, vite fait, m'indiquer la différence entre make_gestion_url et get_gestion_url, stp ?

et je me répond moi même :

fix pour l’envoi de la bonne URL de gestion du rdv dans leur base de dedoublonnage, ça bloque actuellement l’annulation des rdvs.

Je complète : le bug c'est d'utiliser rdv.gestion_url qui n'existe pas forcément, du coté searchApplicationIds c'est fait correctement en utilisant une URL générique vers le hub. Le souci ici c'est que les rdv sont poussés dans la base de dédoublonnage où on a pas accès à request pour créer une URL donc j'ai introduit un setting BASE_URL et j'ai créé des méthodes génériques sur le modèle RendezVous pour créer ces urls (gestion, annulation) et ainsi être certain d'avoir des URLs communes entre push_rdv() et le endpoint searchApplicationIds.

> J'ai pas tout compris (surtout le lien avec le ticket client). > Est-ce que tu pourrais, vite fait, m'indiquer la différence entre make_gestion_url et get_gestion_url, stp ? > > et je me répond moi même : > > fix pour l’envoi de la bonne URL de gestion du rdv dans leur base de dedoublonnage, ça bloque actuellement l’annulation des rdvs. Je complète : le bug c'est d'utiliser rdv.gestion_url qui n'existe pas forcément, du coté searchApplicationIds c'est fait correctement en utilisant une URL générique vers le hub. Le souci ici c'est que les rdv sont poussés dans la base de dédoublonnage où on a pas accès à request pour créer une URL donc j'ai introduit un setting BASE_URL et j'ai créé des méthodes génériques sur le modèle RendezVous pour créer ces urls (gestion, annulation) et ainsi être certain d'avoir des URLs communes entre push_rdv() et le endpoint searchApplicationIds.
bdauvergne merged commit a8553f0c72 into main 2024-04-15 10:55:57 +02:00
bdauvergne deleted branch wip/89529-L-URL-de-gestion-envoye-dans-l-A 2024-04-15 10:55:57 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/ants-hub#16
No description provided.
Delete Branch "wip/89529-L-URL-de-gestion-envoye-dans-l-A"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?