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

Connector/digitalrecruiters #187

Merged
merged 64 commits into from
Nov 23, 2023
Merged

Conversation

Abdellahitech
Copy link
Contributor

Digital Recruiters PR

Abdellahitech and others added 18 commits September 28, 2023 16:14
initial commit of digital recruiters with essentials features (not completed). Actions:
* Pull job lit
* Push profile
* Docs
* Modification about formatting and initialization of the connector
modify readme
update readme
delete some missing
updates pushprofile and pull job readme
update all readme to correct digital recruiters name
create test-config.yaml
add pull profile
feat: add dr to manifest
docs: modify example
docs: add digitalrecruiters to readme
fix: flake8 error
fix: isort issue
docs: update dr docs
docs: update digitalrecruiters README.md
@the-forest-tree
Copy link
Collaborator

the-forest-tree commented Nov 13, 2023

@Abdellahitech Merci pour le travail. C'est bon pour moi.

Si les tests core passent je vais merger incha Lah.

Edit: Finalement encore une peu de travail sur la partie test avant de merder incha Lah

Copy link
Collaborator

Choose a reason for hiding this comment

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

@Abdellahitech En local tu as pu faire tourner ces tests ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

je ne me souviens pas je dois revoir cela.. car j'avais pas une bonne compréhension de ces fichiers yaml donc j'ai suivi un exemple qui a été fait par stef et je ne suis pas sûr que l'execution de cela a marché.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

je peux revoir cette partie, si tu veux

Copy link
Collaborator

Choose a reason for hiding this comment

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

Avant de changer dis moi d'abord. En local quand tu fais tes tests tu as besoin de changer les credentials chaque combien ?
Si la réponse est differente par action de Warehouse donne moi le détail comme ca je peux te dire quoi faire ici incha Lah

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui c'est différent selon type d'action:

  • Pour pull job list et push profile on utilise qu'un seul token fourni par digitalrecruiters.
  • Pour pull profile list nous devons passé par la procédure de génération de token puis du refresh token.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ce que je comprends c'est que le token fourni par Digital Recruiter est fixe tu as pas a le changer toutes les 24h ou truc du genre.
Et pareil le triplet api_key , username et password dans le cas pull_profile_list est fixe.

Dans ce cas voila ce que tu vas faire:

Puis fait en sorte de tester :

  • chacune des Warehouse que tu as contribué
  • chacune des actions du connecteur DigitalRecruiters

Pour cela c'est possible que tu ais besoin de modifier ce qu'il y'a actuellement dans ce fichier. Mais tu as besoin de bien comprendre la doc de cette partie.

Une fois que en local ca marche il faudra que tu me transmette le fichier de secret pour que je le rajoute sur AWS.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ça marche je revois cette documentation inchallah

@the-forest-tree
Copy link
Collaborator

@Abdellahitech Merci pour le travail. C'est bon pour moi.

Si les tests core passent je vais merger incha Lah.

@the-forest-tree the-forest-tree merged commit 0151640 into master Nov 23, 2023
3 of 4 checks passed
Abdellahitech added a commit that referenced this pull request Mar 15, 2024
Abdellahitech added a commit that referenced this pull request Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants