-
Notifications
You must be signed in to change notification settings - Fork 9
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(back): use uuid instead of imported id, for items and objects. keep original id #55
fix(back): use uuid instead of imported id, for items and objects. keep original id #55
Conversation
Some tests are based on id (original data id), so generating a shortuuid id breaks theses tests. |
It is probably best to access the table directly with Lance to retrieve the generated UUIDs. This will allow calling the endpoints in the same way that the frontend does, which is what we want to validate with these tests. |
…ep original id (#55) * fix(back): coco_importer with uuid * fix(back): keep original_id, uuid for objects, all importers * fix(tests): adapt tests to uuid ids * fix(back): adapt tests + lookup in tables with id_list from first table select * fix(test): format * fix(back): if exists, export original_id to id * fix(back): make original_id generic
This is a draft for using shortuuid instead of original id in coco_importer, to ensure id unicity
should fix #50