Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Suppression du champs booléen "is_valid" #223

Closed
DonovanMaillard opened this issue Aug 12, 2021 · 1 comment
Closed

Suppression du champs booléen "is_valid" #223

DonovanMaillard opened this issue Aug 12, 2021 · 1 comment
Labels
BDD About database enhancement New feature or request solved in next release This issue will be fixed in next release

Comments

@DonovanMaillard
Copy link
Collaborator

Historique :
A l'origine, le module d'imports a été conçu comme une "interface d'aide à l'import" destinée aux administrateurs de base. L'idée était que les administrateurs puissent switcher entre le module et la base, et faire des traitements complémentaires en SQL sans tout importer manuellement, car il semblait trop complexe que le module fasse tout à partir du fichier source.
Aujourd'hui avec Ginco notamment et davantage d'utilisateurs, le module a évolué et s'adresse de plus en plus à "tout public", avec un fonctionnement complet et autonome. Les traitements sont faits en une seule vague après laquelle l'utilisateur reçoit un mail, et ajouter des interventions manuelles en cours de process est au mieux source de soucis/conflits...

Dans ce contexte initial, un champs "is_valid" permet au module de déclarer les données invalides (is_valid->false), et à l'administrateur d'en invalider certaines autres. A la fin le module n'importe que ce qui n'est pas false.

Ce champs est devenu moins pertinent, il suffirait de s'appuyer seulement sur "invalid_reason" -> S'il est vide, c'est qu'il n'y a pas d'erreur, et donc que la donnée est valide et importable.

Nous proposons de supprimer le champs is_valid pour limiter le nombre de champs créés et redondants.

@DonovanMaillard DonovanMaillard added BDD About database enhancement New feature or request question Further information is requested labels Aug 12, 2021
@DonovanMaillard DonovanMaillard added solved in next release This issue will be fixed in next release and removed question Further information is requested labels Jan 3, 2022
@DonovanMaillard
Copy link
Collaborator Author

Fait dans la version 1.2.0

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
BDD About database enhancement New feature or request solved in next release This issue will be fixed in next release
Development

No branches or pull requests

1 participant