Admin: Créer une action export_cta dans l’admin [GEN-1659] #4118
Conversation
tonial
changed the title
tonial
force-pushed
the
alaurent/sso_logos
branch
3 times, most recently
from
2921a9e to
5c4577b
Compare
|
Je vais attendre #4099 et suivre la même méthode. NB : peut être aussi essayer de passer le code SQL en django pour une meilleur maintenabilité ? |
tonial
force-pushed
the
alaurent/sso_logos
branch
4 times, most recently
from
cebad99 to
a5e3b0f
Compare
tonial
force-pushed
the
alaurent/sso_logos
branch
7 times, most recently
from
3500f69 to
44e22e4
Compare
|
C'est bon, j'ai ré-utilisé ce que @francoisfreitag a ajouté pour l'export de candidatures inconnues. J'ai laissé l'export CTA à base de requête SQL brute parce que ça marche et qu'à présent c'est testé. |
|
(La CI est rouge 🤔 ) |
tonial
force-pushed
the
alaurent/sso_logos
branch
from
44e22e4 to
7b0625d
Compare
|
EDIT: finalement j'ai passé l'export CTA en ORM, ça permet de réutiliser des fonction python maintenue (pour la région notamment) et c'était rapide à faire |
tonial
force-pushed
the
alaurent/sso_logos
branch
from
7b0625d to
74124e0
Compare
tonial
force-pushed
the
alaurent/sso_logos
branch
from
74124e0 to
336dee4
Compare
tonial
force-pushed
the
alaurent/sso_logos
branch
from
336dee4 to
a4a5ce7
Compare
francoisfreitag
left a comment
francoisfreitag
left a comment
There was a problem hiding this comment.
Très bien, toujours ça de moins à faire pour supportix. Merci !
👍
| buffer = io.BytesIO() | ||
| workbook.save(buffer) | ||
| buffer.seek(0) | ||
|
|
||
| return FileResponse( | ||
| buffer, | ||
| as_attachment=True, | ||
| filename=f"rejets_api_france_travail_{get_export_ts()}.xlsx", | ||
| content_type="application/vnd.openxmlformats-officedocument.spreadsheetml.sheet", | ||
| ) |
There was a problem hiding this comment.
S’assure que close() sera bien appelé en cas d’exception.
| buffer = io.BytesIO() | |
| workbook.save(buffer) | |
| buffer.seek(0) | |
| return FileResponse( | |
| buffer, | |
| as_attachment=True, | |
| filename=f"rejets_api_france_travail_{get_export_ts()}.xlsx", | |
| content_type="application/vnd.openxmlformats-officedocument.spreadsheetml.sheet", | |
| ) | |
| with io.BytesIO() as buffer: | |
| workbook.save(buffer) | |
| buffer.seek(0) | |
| return FileResponse( | |
| buffer, | |
| as_attachment=True, | |
| filename=f"rejets_api_france_travail_{get_export_ts()}.xlsx", | |
| content_type="application/vnd.openxmlformats-officedocument.spreadsheetml.sheet", | |
| ) |
There was a problem hiding this comment.
Ça ne fonctionne pas : le buffer semble se fermer trop tôt, et du coup le fichier fait 0 octets (à la fois en serveur de dev, et dans les tests)
There was a problem hiding this comment.
Effectivement. Bon, laissons comme ça, normalement ça finira bien par être garbage collecté.
| @pytest.mark.parametrize( | ||
| "factory,factory_kwargs,expected_status", | ||
| [ | ||
| (JobSeekerFactory, {}, 404), | ||
| (EmployerFactory, {"with_company": True}, 404), | ||
| (PrescriberFactory, {}, 404), | ||
| (LaborInspectorFactory, {"membership": True}, 404), | ||
| (ItouStaffFactory, {}, 404), | ||
| (ItouStaffFactory, {"is_superuser": True}, 302), # redirects to dashboard if no file | ||
| ], | ||
| ) | ||
| def test_requires_superuser(self, client, factory, factory_kwargs, expected_status): | ||
| user = factory(**factory_kwargs) | ||
| client.force_login(user) | ||
| response = client.get(reverse("itou_staff_views:export_ft_api_rejections")) | ||
| assert response.status_code == expected_status |
There was a problem hiding this comment.
Ça donne envie de générer ce parametrize à partir des URLs déclarés dans itou/www/itou_staff_views/urls.py.
There was a problem hiding this comment.
Je laisse au prochain qui passera par là 🙈
🤔 Pourquoi ?
Faciliter la tâche de Supportix
🍰 Comment ?
Ajouts de liens sur le tableau de bord des administrateurs
💻 Captures d'écran
🚨 À vérifier
🏝️ Comment tester
Aller dans l'admin dans la nouvelle catégorie des exports personnalisés