Skip to content

API: Ajout d'une API pour rechercher les candidatures pour les CD [GEN-1651]#5588

Merged
leo-naeka merged 3 commits into
masterfrom
leo/job-seekers-api
Feb 21, 2025
Merged

API: Ajout d'une API pour rechercher les candidatures pour les CD [GEN-1651]#5588
leo-naeka merged 3 commits into
masterfrom
leo/job-seekers-api

Conversation

@leo-naeka

@leo-naeka leo-naeka commented Feb 11, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

  • Permettre aux CD de suivre les candidatures des bénéficiaires du RSA (peu importe l’émetteur de la candidature)
  • Supprimer les doubles saisies pour les agents du CD

🍰 Comment ?

En mettant une API à disposition de leur éditeur de logiciel

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

@leo-naeka leo-naeka added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog. labels Feb 11, 2025
@leo-naeka leo-naeka self-assigned this Feb 11, 2025
@notion-workspace

notion-workspace Bot commented Feb 11, 2025

Copy link
Copy Markdown

🌿 Création d’une API pour faciliter le suivi des candidatures de BRSA pour les agents du CD 22

@github-actions

github-actions Bot commented Feb 11, 2025

Copy link
Copy Markdown

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

@leo-naeka leo-naeka marked this pull request as ready for review February 11, 2025 19:23
@leo-naeka

leo-naeka commented Feb 11, 2025

Copy link
Copy Markdown
Contributor Author

Comme je vous disais, j'envisage de modifier le dernier test pour instancier des objets stables et comparer avec un snapshot plutôt que faire 36 assertions. Mais je veux bien déjà votre avis sur le reste 🙏

tonial
tonial reviewed Feb 11, 2025
View reviewed changes

@tonial tonial left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ça me semble être la bonne direction

Comment thread itou/api/job_application_api/serializers.py
Comment thread itou/api/job_application_api/serializers.py Outdated
Comment thread itou/api/job_application_api/serializers.py
Comment thread tests/api/job_applications_api/test_search.py Outdated
xavfernandez
xavfernandez reviewed Feb 12, 2025
View reviewed changes

@xavfernandez xavfernandez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Très propre :)

Comment thread itou/api/job_application_api/perms.py Outdated
Comment thread itou/api/job_application_api/schema.py
@leo-naeka leo-naeka force-pushed the leo/job-seekers-api branch 4 times, most recently from 21ccb1d to 61c0b58 Compare February 17, 2025 11:11
@leo-naeka

leo-naeka commented Feb 17, 2025

Copy link
Copy Markdown
Contributor Author

Je relance la relecture, il y a eu des ajustements vus avec Zohra, et la reprise de petites choses évoquées.
Tous ces ajustements sont dans des commits dédiés pour faciliter la review et suivre les changements, ils seront fusionnés dans le 3e commit.

Seule exception, l'ajout d'un test pour s'assurer que les exemples sont raccord avec le sérialiseur courant, sur la très juste remarque de Xavier, qui a été intégré au commit original : e93d29f#diff-b731c845e33c86cb308c08f34b2945ae33f699b584a1771610f4c79a6702797a

xavfernandez
xavfernandez reviewed Feb 19, 2025
View reviewed changes
Comment thread itou/api/job_application_api/views.py
Comment thread itou/api/job_application_api/views.py
tonial
tonial approved these changes Feb 20, 2025
View reviewed changes
Comment thread itou/api/job_application_api/serializers.py
Comment thread itou/api/job_application_api/views.py
Comment thread tests/api/job_applications_api/test_search.py Outdated
Comment thread tests/api/job_applications_api/test_search.py Outdated
Comment thread itou/api/job_application_api/serializers.py Outdated
@tonial

tonial commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

J'ai fait quelques remarques, mais rien que je trouve bloquant :)

@leo-naeka leo-naeka force-pushed the leo/job-seekers-api branch 2 times, most recently from af458cd to 639fafe Compare February 20, 2025 14:56
@leo-naeka

leo-naeka commented Feb 20, 2025

Copy link
Copy Markdown
Contributor Author

Merci à tous les deux. Je prévois de rebase et fusionner tous les commits de fixup d'ici une heure, MEP ensuite.

@leo-naeka leo-naeka added this pull request to the merge queue Feb 21, 2025
Merged via the queue into master with commit c4f228e Feb 21, 2025
@leo-naeka leo-naeka deleted the leo/job-seekers-api branch February 21, 2025 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xavfernandez xavfernandez xavfernandez left review comments
@tonial tonial tonial approved these changes

Assignees

@leo-naeka leo-naeka

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@leo-naeka @tonial @xavfernandez