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

Bug #14108 [Collection] - [Issues with Adding Directory to a Deposit Project] #2414

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Salimdev
Copy link
Contributor

Fix bugs (Incorrect size: When uploading a directory from the file explorer, the displayed size is 0 bytes, Data loss: Several pieces of data are lost after validation.)

In drag & drop and with upload files/directories, handling empty folders (If the empty folder is located inside another parent directory, an error message will be displayed).

@Salimdev Salimdev added the bug Something isn't working label Jan 27, 2025
@Salimdev Salimdev added this to the IT 147 milestone Jan 27, 2025
@Salimdev Salimdev self-assigned this Jan 27, 2025
@Salimdev Salimdev force-pushed the bug_14108 branch 3 times, most recently from 328fa11 to 2f05a8d Compare January 27, 2025 14:46
Copy link
Contributor

@marob marob left a comment

Choose a reason for hiding this comment

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

Si on sélectionne un dossier puis le supprime, le bouton "Suivant" n'est pas grisé.

Il y a un refactoring global à prévoir. Un FileSelector ne devrait pas avoir un zip en input. D'ailleurs, il ne devrait peut-être pas gérer de zip tout court. En tout cas, il doit pouvoir fonctionner sans zip (usages autres que pour collecte).

Il faut écrire des tests unitaires.

@Salimdev Salimdev force-pushed the bug_14108 branch 2 times, most recently from 48eb3c0 to 015173c Compare January 29, 2025 13:43
@ProgrammeVitam ProgrammeVitam deleted a comment from marob Feb 3, 2025
@Salimdev Salimdev requested a review from marob February 3, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants