-
Notifications
You must be signed in to change notification settings - Fork 81
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
Import CSV 2.0 #2310
Comments
|
The UI, we have some general UX questions that we have included in #2353, or maybe everything should be consolidated here? |
lets finish the the more tech side here and continue with ux things in the other one.
should a blank value on the file column delete the file then ? |
@txau thinking about this
what if they want to add a csv as an attachment, should we some kind of naming convention to specify which csv to use for the import ? |
@daneryl good call. It totally makes sense. |
@txau, so do nothing about it until we implement attachments sounds good ? |
@daneryl since you are going to check nevertheless for the presence of a CSV, check for a particular CSV name, ie. import.csv. |
@daneryl when you have a .zip file with multiple files, including multiple .csv files, it correctly imports the If there's no It stays on the sidebar indefinitely. You can close it, but it reappears when you click the "import" button. Which means you lose access to the import form, until you refresh the page. The notification goes on the top bar when you go to the library page: I can't see any way to close it. It stays there indefinitely, until you refresh the page. |
@habbes ok, nice one ... will look into it thx ! |
@habbes the fix is ready on the PR, error handling was missing on the endpoint, ready for QA |
@daneryl alright And regarding this:
How do I add attachments in the zip for imports? |
@habbes attachments support is not implemented yet, this was just to prevent this problem in the future. |
@daneryl oh alright |
Add an icon for importing thesauri in a thesauriThe text was updated successfully, but these errors were encountered: