Skip to content

Conversation

@francoisfreitag
Copy link
Member

🤔 Pourquoi ?

Éviter les confusions.

💻 Captures d'écran

🏝️ Comment tester

  1. Se connecter en tant que l’EI
  2. Depuis le dropdown du « Mon espace », aller sur « Accès aux API »
  3. Voir le message d’avertissement à destination des utilisateurs de l’API employee-records

The applicants API does not expose the SIRET.
@francoisfreitag francoisfreitag added the modifié Modifié dans le changelog. label May 7, 2024
@francoisfreitag francoisfreitag self-assigned this May 7, 2024
@francoisfreitag francoisfreitag requested a review from tonial May 7, 2024 08:05
@tonial
Copy link
Contributor

tonial commented May 7, 2024

Il faut remplacer SIRET par "uid_structures"

Important : Votre éditeur de logiciel doit filtrer les données par “uid_structures" afin qu’elles soient rattachées dans les bonnes structures.

@francoisfreitag
Copy link
Member Author

Ah bon ? Pour moi, l’uid_structure n’est nécessaire que si le mode multi_structure est activé, auquel cas l’employeur devrait s’attendre à ce que les candidats de plusieurs structures soient listées, et éventuellement filter les résultats. D’autant que si seule une structure l’intéresse, il peut directement utiliser uid_structures=.

En revanche, pour l’API employee-records, il n’est pas évident que les fiches salarié de toutes les structures sont présentes dans les résultats.

@tonial
Copy link
Contributor

tonial commented May 7, 2024

Tu as raison pour les 2 points.
Ma remarque tenait du fait que la carte notion indiquait ce filtre, et que j'ai mal renommé celui-ci.
Mais finalement, il ne sert peut être pas à grand chose, et ta proposition semble plus pertinente 👍

@tonial tonial closed this May 7, 2024
@tonial tonial reopened this May 7, 2024
@tonial
Copy link
Contributor

tonial commented May 7, 2024

Oups, j'ai réussi à clore la PR sans le vouloir....

@francoisfreitag francoisfreitag added this pull request to the merge queue May 8, 2024
Merged via the queue into master with commit 328b016 May 8, 2024
@francoisfreitag francoisfreitag deleted the ff/api-hint branch May 8, 2024 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants