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

Rendre plus résiliente la route de reorder des ressources #1630

Open
maudetes opened this issue Jan 8, 2025 · 1 comment · May be fixed by opendatateam/udata#3243
Open

Rendre plus résiliente la route de reorder des ressources #1630

maudetes opened this issue Jan 8, 2025 · 1 comment · May be fixed by opendatateam/udata#3243

Comments

@maudetes
Copy link
Contributor

maudetes commented Jan 8, 2025

Lors d'un reorder des ressources d'un dataset, la route est très permissive.
On lui passe en body une liste de ressources : ["ae2b016a-014f-4f9e-a4cf-e28d193f19be","a10e29ff-81ff-4e3a-9dd4-b8ebcf2d3bed","80783035-d4f9-429b-a8ab-d52aa90e53c3","80783035-d4f9-429b-a8ab-d52aa90e53c3"].

  1. On peut par exemple mettre une ressource en doublon ou triple, ce qui amène à sa duplication (voir https://mattermost.incubateur.net/betagouv/pl/7wz5rrczmtgtzk6yuf15xfqwge).
  2. On peut aussi enlever une ressource, qui va donc disparaître du dataset, même si le fichier existe toujours sur les serveurs.
  3. Que se passe-t-il s'il s'agit d'une ressource d'un autre JDD ?
  4. Faut-il prévoir une migration pour des cas de duplicats de ressources existants au sein d'un JDD ?
@maudetes maudetes changed the title Lors d'un reorder des ressources d'un dataset si on met deux fois le même id d'une ressource, elle est dupliquée en tout point Rendre plus résilient la route de reorder des ressources Jan 9, 2025
@maudetes maudetes changed the title Rendre plus résilient la route de reorder des ressources Rendre plus résiliente la route de reorder des ressources Jan 9, 2025
@Pierlou
Copy link

Pierlou commented Jan 9, 2025

J'ai commencé un draft d'ajouts de garde-fous : opendatateam/udata#3243

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 📝 Todo
Development

Successfully merging a pull request may close this issue.

2 participants