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

Permettre d'importer des aidants/responsables en donnant le numéro de demande datapass de l'orga à la place de l'id interne Django #466

Merged
merged 5 commits into from
Nov 22, 2021

Conversation

tut-tuuut
Copy link
Member

🌮 Objectif

Faire gagner BEAUCOUP de temps aux bizdev lors de la création des comptes des responsables à partir de l'export datapass.

Genre, BEAUCOUP. Pour l'instant ils-elles doivent récupérer les id à la main dans Django…

🔍 Implémentation

  • Utilisation du widget ForeignKeyWidget fourni par le module django-import-export.
  • Oui, c'est tout. J'ai un peu honte d'avoir loupé ça dans la doc la dernière fois.

Comment on lines 11 to 12
<dd>Numéro de demande datapass de l'organisation. Si vous donnez ce numéro,
<code>organisation_id</code> n'est pas obligatoire.</dd>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<dd>Numéro de demande datapass de l'organisation. Si vous donnez ce numéro,
<code>organisation_id</code> n'est pas obligatoire.</dd>
<dd>Numéro de demande datapass de l'organisation. Si vous donnez ce numéro,
<code>organisation_id</code> n'est pas obligatoire.
</dd>

Déso, ça me titille 😅

Copy link
Member Author

Choose a reason for hiding this comment

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

Effectivement la mise en forme automatique de pycharm est d'accord avec toi. x)

@tut-tuuut
Copy link
Member Author

Désolée pour la double relecture, mais je viens de push un nouveau commit qui sera intéressant fonctionnellement (puisque l'import excel ne va plus servir que pour les responsables).

)
respo_de_datapass_id = Field(
attribute="responsable_de",
widget=ManyToManyWidget(Organisation, field="data_pass_id", separator=";"),
Copy link
Member Author

Choose a reason for hiding this comment

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

Pourquoi un séparateur point-virgule ? Parce que sinon le couple excel+Widget interprète les valeurs de type 4,6 comme "quatre virgule six" au lieu de "quatre et six".

@tut-tuuut tut-tuuut force-pushed the ahr/add-columns-in-hab-requests-exports branch from 6050f82 to beb11e7 Compare November 22, 2021 11:53
Base automatically changed from ahr/add-columns-in-hab-requests-exports to main November 22, 2021 13:22
@tut-tuuut tut-tuuut force-pushed the ahr/import-aidants-with-datapass-id branch from ce2fb9f to 8c063e4 Compare November 22, 2021 15:29
@tut-tuuut tut-tuuut merged commit bd029e7 into main Nov 22, 2021
@tut-tuuut tut-tuuut deleted the ahr/import-aidants-with-datapass-id branch November 22, 2021 17:30
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.

3 participants