-
Notifications
You must be signed in to change notification settings - Fork 0
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
fix: dépôt vecteur - échec à la vérification standard et vecteur #579 #606
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sur la forme je pense qu'il est préférable de coller au vocabulaire de l'API et utiliser Upload
plutôt que Delivery
. Le terme Delivery
n'est d'ailleurs pas présent dans la base de code avant cette PR.
Pourrait-tu renommer les composants, méthodes et fichiers concernés ?
Et en termes d'organisation, je trouve qu'il y a beaucoup de choses dans assets/entrepot/pages/stored_data/StoredDataDetails
dont la moitié concerne les détails d'un Upload
. Il vaudrait peut-être mieux réorganiser en un dossier assets/pages/details/
avec des sous-dossiers stored_data
et upload
. @ocruze tu as un avis ?
Enfin voyez ensemble pour résoudre les conflits avec la branche main
.
assets/entrepot/pages/datasheet/DatasheetView/DatasetListTab/UnfinishedUploadList.tsx
Show resolved
Hide resolved
assets/entrepot/pages/stored_data/StoredDataDetails/DeliveryDetails.tsx
Outdated
Show resolved
Hide resolved
Je suis d'accord pour renommer Delivery en Upload. Pour la réorganisation de fichiers, je pense qu'il n'y pas assez de fichiers différents pour stored_data et upload pour faire des sous-dossiers. Seulement les composants PreviewTab et StoredDataDetails ont 2 composants séparés, les autres composants (tout ce qui est dans ReportTab) sont communs. Ce que je propose, c'est d'avoir comme dossier principal
|
Co-authored-by: Orka Arnest CRUZE <33525693+ocruze@users.noreply.github.com>
Co-authored-by: Orka Arnest CRUZE <33525693+ocruze@users.noreply.github.com>
Co-authored-by: Orka Arnest CRUZE <33525693+ocruze@users.noreply.github.com>
7a71b74
to
4bab95e
Compare
contient les issues #582, #583, #584, #585 et #588.