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

feat: sélection par défaut des enjeux #267

Merged
merged 2 commits into from
Feb 12, 2025
Merged

Conversation

ikarius
Copy link
Collaborator

@ikarius ikarius commented Feb 7, 2025

Voir : https://trello.com/c/xxZE6tQp/368-passer-la-s%C3%A9lection-des-enjeux-%C3%A0-s%C3%A9lectionn%C3%A9-par-d%C3%A9faut

Tous les enjeux sont sélectionnés par défaut lors de la création d'un
rapport CSRD.

note :
les anciens rapports CSRD gardent pour l'instant l'ancien
comportement.
A voir si une migration de données est nécessaire : à confirmer côté métier.

@ikarius ikarius self-assigned this Feb 7, 2025
@ikarius
Copy link
Collaborator Author

ikarius commented Feb 12, 2025

Selon Claudia manque encore des choses, je vois si c'est une PR séparé ou pas.

@ikarius ikarius added the technique Demande ou problème technique label Feb 12, 2025
Copy link
Collaborator

@sblondon sblondon left a comment

Choose a reason for hiding this comment

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

Juste un détail mais sinon c'est bon pour moi.

Reste à voir si le champ de justification de la non sélection finit dans une autre branche et PR

impact/reglementations/tests/test_csrd_views.py Outdated Show resolved Hide resolved
Tous les enjeux sont sélectionnés par défaut lors de la création d'un
rapport CSRD.

note: les anciens rapports CSRD gardent pour l'instant l'ancien
comportement. A voir si une migration est nécessaire.
@ikarius ikarius force-pushed the enjeux-selectionnes-par-defaut branch from 1c83884 to e400aa7 Compare February 12, 2025 16:48
@ikarius ikarius merged commit d1c2cf4 into main Feb 12, 2025
5 checks passed
@ikarius ikarius deleted the enjeux-selectionnes-par-defaut branch February 12, 2025 16:51
@ikarius ikarius added the amélioration Nouvelle fonctionnalité ou demande label Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amélioration Nouvelle fonctionnalité ou demande technique Demande ou problème technique
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants