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

Allow setting the order of files for records #2573

Open
max-moser opened this issue Feb 9, 2024 · 2 comments
Open

Allow setting the order of files for records #2573

max-moser opened this issue Feb 9, 2024 · 2 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@max-moser
Copy link
Contributor

Overview

It's been recently requested to change the order of files for a record because the default order is somewhat arbitrary (definitely not lexicographic).
InvenioRDM seems to have some basic support for this (record.files.order), but setting this field results in internal server errors during the lifecycle of the record/draft.

Problem

For example, I've set the order of files for a draft through invenio shell, and after draft.commit() and during rdm_rec_svc.indexer.reindex(draft), I get the following exception:
RequestError: RequestError(400, 'strict_dynamic_mapping_exception', 'mapping set to strict, dynamic introduction of [order] within [files] is not allowed')

Optimal solution

It would be nice to have the order of appearance for files be changeable by the user, maybe through drag & drop on the deposit page.

Alternatives

Basically it would already suffice to support setting the file order through the backend.

@max-moser max-moser added bug Something isn't working enhancement New feature or request labels Feb 9, 2024
@tmorrell
Copy link
Contributor

tmorrell commented Feb 9, 2024

Our users would also be very happy if file reordering was possible

Copy link
Contributor

This issue was automatically marked as stale.

@github-actions github-actions bot added the stale No activity for more than 60 days. label Apr 10, 2024
@tmorrell tmorrell removed the stale No activity for more than 60 days. label Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants