entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

toulouse-maelis: endpoint pour faire la recherche d'usagers depuis le portail agent (#72310) #88

Merged
nroche merged 2 commits from wip/72310-parsifal-search-endpoints into main 2023-02-15 16:32:42 +01:00
Conversation 1 Commits 2 Files Changed 4 +330
nroche commented 2023-02-04 19:35:07 +01:00
Owner
Copy Link

Maj du WSDL pour les tests (il y a un ajout au DUI que je traiterai dans un autre ticket)
Ajout d'un endpoint pour appeler un WS de recherche (plus ou moins) générique.
Ajout d'un second enpoint vers un WS supplémentaire pour rechercher les dossier depuis le numéro DUI, parce que le premier WS ne le fait pas.

Mon idée est d'avoir ces 2 moteurs d'indexation dans la cellule recherche du portail agent.

Maj du WSDL pour les tests (il y a un ajout au DUI que je traiterai dans un autre ticket) Ajout d'un endpoint pour appeler un WS de recherche (plus ou moins) générique. Ajout d'un second enpoint vers un WS supplémentaire pour rechercher les dossier depuis le numéro DUI, parce que le premier WS ne le fait pas. Mon idée est d'avoir ces 2 moteurs d'indexation dans la cellule recherche du portail agent.
nroche force-pushed wip/72310-parsifal-search-endpoints from f49d1ceca0 to 09e167d0ee 2023-02-06 18:20:37 +01:00 Compare
nroche force-pushed wip/72310-parsifal-search-endpoints from 09e167d0ee to 584e1732bf 2023-02-12 21:21:04 +01:00 Compare
nroche force-pushed wip/72310-parsifal-search-endpoints from 584e1732bf to 336cfc4bb5 2023-02-14 20:26:28 +01:00 Compare
aberriot reviewed 2023-02-15 08:54:06 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -863,0 +871,4 @@
)
def search_family(self, request, q=None):
data = []
if q and len(q) >= 4:
aberriot commented 2023-02-15 08:49:45 +01:00
Owner
Copy Link

Quel est l'enjeu de ce minimum de 4 caractères ? ça ne risque pas d'être très pénible pour des recherches à 3 caractères ou moins, par exemple sur un nom de famille court ?

Quel est l'enjeu de ce minimum de 4 caractères ? ça ne risque pas d'être très pénible pour des recherches à 3 caractères ou moins, par exemple sur un nom de famille court ?
nroche commented 2023-02-15 11:04:53 +01:00
Author
Owner
Copy Link

C'est purement empirique : si j'envoie un mot trop court, Maélis mouline longtemps puis me renvoie tous les DUI de sa base.
Ici il faudrait que ce soit Maélis qui ne réponde rien tant que le texte saisi n'est pas suffisamment explicite pour cibler un compte et je ne devrais pas ajouter cette astuce "métier" dans le connecteur, tu as raison.
Dans les faits, avant que Sigec ne change quoi que ce soit, je préfère avoir quelque-chose qui fonctionne, tant que ça ne tient que sur une ou 2 lignes de code.
(Maélis recherche sur le nom et/ou le prénom via le WS readFamilyListFromFullName,
donc pour rechercher monsieur WU, l'agent devra ajouter son prénom dans la recherche)

C'est purement empirique : si j'envoie un mot trop court, Maélis mouline longtemps puis me renvoie tous les DUI de sa base. Ici il faudrait que ce soit Maélis qui ne réponde rien tant que le texte saisi n'est pas suffisamment explicite pour cibler un compte et je ne devrais pas ajouter cette astuce "métier" dans le connecteur, tu as raison. Dans les faits, avant que Sigec ne change quoi que ce soit, je préfère avoir quelque-chose qui fonctionne, tant que ça ne tient que sur une ou 2 lignes de code. (Maélis recherche sur le nom et/ou le prénom via le WS readFamilyListFromFullName, donc pour rechercher monsieur WU, l'agent devra ajouter son prénom dans la recherche)
aberriot approved these changes 2023-02-15 11:09:37 +01:00
aberriot left a comment
Owner
Copy Link

C'est purement empirique : si j'envoie un mot trop court, Maélis mouline longtemps puis me renvoie tous les DUI de sa base.

Okay, je me doutais que c'était un truc du genre. Bon si le souci est upstream, difficile de faire beaucoup mieux, à part éventuellement rajouter un commentaire pour expliquer ça ?

> C'est purement empirique : si j'envoie un mot trop court, Maélis mouline longtemps puis me renvoie tous les DUI de sa base. Okay, je me doutais que c'était un truc du genre. Bon si le souci est upstream, difficile de faire beaucoup mieux, à part éventuellement rajouter un commentaire pour expliquer ça ?
nroche force-pushed wip/72310-parsifal-search-endpoints from 336cfc4bb5 to bf7ccc253a 2023-02-15 12:14:35 +01:00 Compare
nroche force-pushed wip/72310-parsifal-search-endpoints from bf7ccc253a to 2382293821 2023-02-15 12:15:35 +01:00 Compare
nroche force-pushed wip/72310-parsifal-search-endpoints from 2382293821 to ab55213ed7 2023-02-15 12:35:37 +01:00 Compare
nroche merged commit ab55213ed7 into main 2023-02-15 16:32:42 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
aberriot
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/passerelle#88
No description provided.
Delete Branch "wip/72310-parsifal-search-endpoints"

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?