Add logic for pulling profiles and parse resumes in HFW #148
Conversation
There was a problem hiding this comment.
Merci @riquelme222 pour le travail !
Il y'a quelques modifications a apporter et aussi corriger les erreurs flake8. Si tu check la pipeline tu vas les voir.
Autrement en local tu peux faire make flake8 pour les voir.
Tu peux aussi run make style normalement c'est censé corriger une partie des erreurs possibles.
Aussi si pour acceder à l'API TeamTailor il n'y a de white listing ou truc du genre. Essaye de rajouter ce qu'il faut pour tester ta nouvelle action de manière automatique.
| def get_authorization(self) -> str: | ||
| return f"Token token={self.api_token}" |
There was a problem hiding this comment.
Ici deux remarques:
-
Mon ressenti c'est que c'est pas la peine de rajouter une méthode au objects du type
XxxxParameters. Pour que ca reste focus sur leur but initial. Cad transmettre de manière simple et directe les paramètres nécéssaires. Sauf si c'est déjà utilisé dans d'autreXxxxxParameters. De mémoire dans ce que j'ai fais moi ca ne devrait pas être le cas wa Allah a3lam -
Sinon de manière général je pense qu'une property aurait été mieux si y'avait pas la première remarque. Vu que tu donne aucun paramètre. Ca permet de faire
parameters.get_authorizationplutot queparameter.get_authorization()et dans ce cas encore mieuxparameters.authorization
| else: | ||
| return None |
There was a problem hiding this comment.
Ici pas besoin de else vu que tu return dans le if
if response.ok:
return response.content
return None Aussi est ce que juste return None de manière silencieuse c'est le mieux ?
Pourquoi tu raises pas une exception pour que ca remonte ?
There was a problem hiding this comment.
Par rapport à la deuxième question je pense qu'une solution safe au début quand on sait pas trop comment réagi l'API en face c'est de plutot raiser.
Comme ca tu es sur que si il y'a erreur et que tu arrives pas download le resume tu auras une erreur qui est plus susceptible d'être remarqué.
Si ca se produit alors la y'aura plus de savoir sur le comportement du download des resumes et eventuellement ca permet de faire une gestion plus fine des erreurs.
| # Get candidate resume using the id since each resume | ||
| # Link is signed and available for 30sec only |
There was a problem hiding this comment.
👍 pour le commentaire information important effectivement
| } | ||
|
|
||
| page = 1 | ||
| while True: |
There was a problem hiding this comment.
En fait ici je pense qu'il faut rajouter un paramètre LIMIT.
Avec un LIMIT par default et qui peut être setté via XxxxParameters
Pour le mode incrémentale c'est important. Histoire de controller combien on pull à chaque itération et éviter que ca tourne plus qu'il ne faut.
Maintenant je pense que le paramètre doit être nullable aussi. Pour que par exemple l'utilisateur en local qui veut utiliser le connecteur pour tout pull d'un coup puisse aussi le faire.
| if candidate_resume_url: | ||
| binary_resume_content = download_file(candidate_resume_url) | ||
|
|
||
| # TODO : this can be put outside in format function |
There was a problem hiding this comment.
Oui pour le coup je pense que ca doit être faire comme tu dis dans le commentaire.
Tu peux faire comme l'action applicant_new du connecteur Talentsoft qui écrit vers la même Warehouse de destination et mettre une fonction format par default qui fait le travail.
No description provided.