Admin: ajout d'un bouton pour transférer des données entre entreprises [GEN-1527] by xavfernandez · Pull Request #4676 · gip-inclusion/les-emplois

xavfernandez · 2024-09-06T15:42:12Z

🤔 Pourquoi ?

Éviter de passer par supportix et avoir plus de logs concernant les transferts.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

notion-workspace · 2024-09-06T15:42:22Z

Ajouter un bouton dans l’admin pour automatiser move_company_data

xavfernandez · 2024-10-01T16:18:49Z

itou/templates/admin/companies/change_company_form.html

+
+{% block object-tools-items %}
+
+    {% if has_change_permission %}


J'hésite à n'autoriser que les superuser dans un premier temps ?

plus sage peut être pendant quelques temps ?

Ça me semble également une bonne idée.

github-actions · 2024-10-04T09:07:06Z

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

tonial

Ça m'a l'air très bien :)

Le code est pas facile à relire par contre ^^' (et n'a pas du être facile à écrire non plus !)

tonial · 2024-10-04T08:53:02Z

itou/templates/admin/companies/change_company_form.html

+
+{% block object-tools-items %}
+
+    {% if has_change_permission %}


plus sage peut être pendant quelques temps ?

francoisfreitag

C’est top ! J’ai hâte que ça soit disponible pour le support. Se brancher dans l’admin n’est peut-être pas le plus pratique, tu aurais pu ajouter la page aux staff views. Mais maintenant que c’est comme ça... 🤷

itou/companies/admin.py

xavfernandez · 2024-11-04T13:40:36Z

itou/companies/transfer.py

+        models.Company.objects.filter(pk=from_company.pk).update(
+            block_job_applications=True,
+            job_applications_blocked_at=timezone.now(),
+            is_searchable=False,


@rsebille J'ai bien gardé la désactivation de la recherche sur la from_company mais je préfère ne pas garder l'activation sur la to_company.

rsebille

Les commits ajoutés me semblent bien 👍.

itou/companies/transfer.py

notion-workspace · 2025-03-17T10:47:31Z

Supportix : automatiser move_company_data

xavfernandez self-assigned this Sep 6, 2024

xavfernandez force-pushed the xfernandez/admin_move_company_data branch 2 times, most recently from 3d5ff2e to 64a3fa1 Compare September 9, 2024 06:58

xavfernandez force-pushed the xfernandez/admin_move_company_data branch 2 times, most recently from 63bbf8b to b297307 Compare October 1, 2024 14:26

xavfernandez added the ajouté Ajouté dans le changelog. label Oct 1, 2024

xavfernandez force-pushed the xfernandez/admin_move_company_data branch 5 times, most recently from 88d0123 to 216d238 Compare October 1, 2024 16:17

xavfernandez marked this pull request as ready for review October 1, 2024 16:18

xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Oct 1, 2024

xavfernandez requested review from francoisfreitag and tonial October 1, 2024 16:18

xavfernandez commented Oct 1, 2024

View reviewed changes

xavfernandez added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Oct 1, 2024

tonial added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Oct 4, 2024

tonial approved these changes Oct 4, 2024

View reviewed changes

francoisfreitag approved these changes Oct 9, 2024

View reviewed changes

itou/companies/admin.py Outdated Show resolved Hide resolved

itou/companies/admin.py Outdated Show resolved Hide resolved

itou/companies/admin.py Outdated Show resolved Hide resolved

itou/companies/admin.py Outdated Show resolved Hide resolved

xavfernandez force-pushed the xfernandez/admin_move_company_data branch 4 times, most recently from 550b9bc to 98cf553 Compare October 17, 2024 12:16

xavfernandez force-pushed the xfernandez/admin_move_company_data branch 2 times, most recently from c645eb6 to 7276b1b Compare November 4, 2024 13:38

xavfernandez requested a review from rsebille November 4, 2024 13:38

xavfernandez commented Nov 4, 2024

View reviewed changes

xavfernandez force-pushed the xfernandez/admin_move_company_data branch from 7276b1b to 97df265 Compare November 13, 2024 11:15

rsebille approved these changes Nov 13, 2024

View reviewed changes

itou/companies/transfer.py Show resolved Hide resolved

companies: add logs to move_company_data & allow its use in the admin

ff9c7c3

xavfernandez force-pushed the xfernandez/admin_move_company_data branch from a0bd5d6 to ff9c7c3 Compare November 14, 2024 08:46

xavfernandez added this pull request to the merge queue Nov 14, 2024

Merged via the queue into master with commit 52ad221 Nov 14, 2024

xavfernandez deleted the xfernandez/admin_move_company_data branch November 14, 2024 09:06


		{% block object-tools-items %}

		{% if has_change_permission %}

Conversation

xavfernandez commented Sep 6, 2024

🤔 Pourquoi ?

🍰 Comment ?

🚨 À vérifier

🏝️ Comment tester

💻 Captures d'écran

Uh oh!

notion-workspace bot commented Sep 6, 2024

Uh oh!

xavfernandez Oct 1, 2024

Choose a reason for hiding this comment

Uh oh!

tonial Oct 4, 2024

Choose a reason for hiding this comment

Uh oh!

francoisfreitag Oct 9, 2024

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Oct 4, 2024

Uh oh!

tonial left a comment

Choose a reason for hiding this comment

Uh oh!

tonial Oct 4, 2024

Choose a reason for hiding this comment

Uh oh!

francoisfreitag left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

xavfernandez Nov 4, 2024

Choose a reason for hiding this comment

Uh oh!

rsebille left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

notion-workspace bot commented Mar 17, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants