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

API Entreprise forms scopes: add more scopes #199

Merged
merged 3 commits into from
May 21, 2024

Conversation

skelz0r
Copy link
Member

@skelz0r skelz0r commented May 21, 2024

Pairing with Charlotte

@skelz0r skelz0r self-assigned this May 21, 2024
Copy link
Contributor

@DorineLam DorineLam left a comment

Choose a reason for hiding this comment

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

Super que vous ayez pu avancer sur le sujet @Charlottecho @skelz0r, mes retours au fil du code + voici le résumé en deux points de mes interrogations :

  • Formulaire libre :
    • Par défaut jusqu'ici on ne cochait pas les données sensibles d'effectif urssaf, de bilans bdf et de liasses fiscales, afin de pousser les usagers à nous fournir un cadre juridique précis. Maintenant que la V2 propose plus de fonctionnalités, on pourrait imaginer montrer/mettre en avant ces données, sans les précocher pour autant.
    • Pour les données protégées qui concernent un périmètre spécifique d'entreprise (par exemple les attestations travaux publics, j'aurai tendance à dire qu'il faut faire comme pour effectif, bilans bdf etc, c'est à dire, les montrer mais pas les cocher.
  • Formulaires pré-remplis où ces données sont cochées par défaut, je pense qu'il faut enrichir le cadre juridique d'une mention spécifiquement pour ces données.

@@ -46,7 +46,11 @@ shared:
scopes:
- unites_legales_etablissements_insee
- associations_djepva
- effectifs_urssaf
Copy link
Contributor

Choose a reason for hiding this comment

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

Pour effectif, bilans bdf et liasses fiscales, on avait fait expres de pas les cocher car faut avoir un cadre juridique précis.

En gros :

  • C'est autorisé, donc faut le faire apparaître
  • Mais ça n'est pas précoché

Copy link
Contributor

Choose a reason for hiding this comment

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

Mandataires sociaux idem, mais je pense que c'est moins "touchy", donc peut-être précoché direct ?

@@ -69,11 +73,17 @@ shared:
scopes:
- unites_legales_etablissements_insee
- associations_djepva
- effectifs_urssaf
Copy link
Contributor

Choose a reason for hiding this comment

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

ici idem que pour marchés publics :
Pour effectif, bilans bdf ET liasses fiscales (commentée ici), on avait fait exprès de pas les cocher car faut avoir un cadre juridique précis.

En gros :

  • c'est autorisé, donc faut le faire apparaître
  • Mais ça n'est pas précoché

- attestation_fiscale_dgfip
- attestation_sociale_urssaf
- cotisations_msa
- cotisations_probtp
Copy link
Contributor

Choose a reason for hiding this comment

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

Ici même dans nos cas d'usages c'est pas si clair, je pense que l'idée c'était de dire, cette donnée n'est délivrée que si vraiment utile.
Donc visible mais décochée car :

  • Probtp et cnetp c'est que pour le travaux publics
  • Qualiopi c'est que pour de l'aide de formation.

@@ -132,9 +143,17 @@ shared:
scopes:
- unites_legales_etablissements_insee
- associations_djepva
- effectifs_urssaf
Copy link
Contributor

Choose a reason for hiding this comment

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

idem que pour marchés publics et aides publiques

- attestation_fiscale_dgfip
- attestation_sociale_urssaf
- cotisations_msa
- cotisations_probtp
Copy link
Contributor

Choose a reason for hiding this comment

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

idem que pour aides publiques

@@ -282,11 +321,14 @@ shared:
- effectifs_urssaf
- mandataires_sociaux_infogreffe
- chiffre_affaires_dgfip
- bilans_bdf
Copy link
Contributor

Choose a reason for hiding this comment

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

Ici si on ajoute ces données par défaut, est-ce qu'il ne faut pas demander un cadre juridique plus précis aussi ?
idem pour effectifs d'ailleurs

scopes:
- unites_legales_etablissements_insee
- associations_djepva
- effectifs_urssaf
Copy link
Contributor

Choose a reason for hiding this comment

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

demander cadre juridique précis ?

@@ -361,11 +412,14 @@ La plateforme, mise en place pour permettre le dépôt de ces aides publiques et
- effectifs_urssaf
- mandataires_sociaux_infogreffe
- chiffre_affaires_dgfip
- bilans_bdf
Copy link
Contributor

Choose a reason for hiding this comment

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

Ici si on ajoute ces données par défaut, est-ce qu'il ne faut pas demander un cadre juridique plus précis aussi ?
idem pour effectifs d'ailleurs

@@ -388,18 +442,6 @@ La plateforme, mise en place pour permettre le dépôt de ces aides publiques et
Afin de faciliter cette investigation du marché fournisseurs, ATEXO propose le module de Sourcing. Combiné au profil acheteur MPE, il permet aux entreprises de mettre en avant leurs informations.

Basé sur l’open data et les renseignements des entreprises, ce module présente aux acheteurs des fiches fournisseurs complètes : établissements, éléments financiers, contrats publics, références significatives, expertises, contacts qualifiés, etc.
scopes:
Copy link
Contributor

Choose a reason for hiding this comment

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

Pourquoi avoir supprimé tous les scopes ?

scopes:
- unites_legales_etablissements_insee
- associations_djepva
- effectifs_urssaf
Copy link
Contributor

Choose a reason for hiding this comment

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

Cadre juridique précis ?

@skelz0r skelz0r force-pushed the api-entreprise/more-scopes branch from 928fa28 to 957a248 Compare May 21, 2024 16:19
@skelz0r skelz0r merged commit fc5f890 into develop May 21, 2024
7 checks passed
@skelz0r skelz0r deleted the api-entreprise/more-scopes branch May 21, 2024 16:31
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