add text orientation property to transcriptions (editable in admin)

closes #850 (closed) #851 (closed) #852 (closed) #853 (closed) #854 (closed) #855 (closed) kind of accidentally because I started fixing the tests that broke

requested review from @erwanrouchet

assigned to @mlbonhomme

marked this merge request as draft

added 1 commit

• 77456ae6 - fix schema issue + have text orientation in bulk endpoint too

Compare with previous version

added 16 commits

• 77456ae6...f8aa3e8f - 14 commits from branch master
• fb293416 - add text orientation property to transcriptions (editable in admin)
• 1c7023ed - fix schema issue + have text orientation in bulk endpoint too

Compare with previous version

marked this merge request as ready

added 1 commit

• 1e21e475 - remove required=False from transcription serializer

Compare with previous version

added 1 commit

• 09050dc2 - add missing default value on some endpoints

Compare with previous version

changed the description

added 8 commits

• 09050dc2...d0af1cf1 - 4 commits from branch master
• 32d583fc - add text orientation property to transcriptions (editable in admin)
• 5ae32482 - fix schema issue + have text orientation in bulk endpoint too
• b6513b26 - remove required=False from transcription serializer
• d1360c28 - add missing default value on some endpoints

Compare with previous version

This is missing some unit tests:

• Test that a user can use CreateTranscription to send an orientation
• Test that CreateTranscription without any orientation uses the default value (probably already verified by existing tests)
• Test that CreateTranscription with an invalid orientation returns an error
• Test that a user can use CreateElementTranscriptions to send an orientation
• Test that CreateElementTranscriptions without any orientation uses the default value (probably already verified by existing tests)
• Test that CreateElementTranscriptions with an invalid orientation returns an error
• Test that PartialUpdateTranscription can be used to update the orientation
• Test that PartialUpdateTranscription with an invalid orientation returns an error
• Test that PartialUpdateTranscription without any orientation does not change the orientation (probably already verified by existing tests)
• Test that UpdateTranscription can update a transcription's orientation
• Test that UpdateTranscription with an invalid orientation returns an error

added 2 commits

• cdf385c7 - WIP: unit tests
• 9e34f4ec - update tests

Compare with previous version

marked this merge request as draft from cdf385c7

resolved all threads

marked this merge request as ready

Let's stop here, this is WAY too large to review. This work needs to be splitted into several smaller merge requests, one per issue.

closed