Skip to content
Snippets Groups Projects

Return a HTTP 400 for too long names using CreateTranscriptionEntities and CreateEntity

Merged Valentin Rigal requested to merge trasncription-entities-500 into release-1.7.0
All threads resolved!

Closes #1876 (closed)

I was getting django.db.utils.OperationalError, so I catched both.

Merge request reports

Checking pipeline status.

Approval is optional

Merged by Erwan RouchetErwan Rouchet 4 months ago (Dec 9, 2024 9:11am UTC)

Merge details

  • Changes merged into release-1.7.0 with 4bc68933 (commits were squashed).
  • Deleted the source branch.
  • Auto-merge enabled

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Erwan Rouchet
  • Could you do the same error handling in the bulkn't endpoint CreateEntity? Various workers and scripts do use it, so they could also hit the same issue.

  • Erwan Rouchet requested changes

    requested changes

  • changed milestone to %Arkindex 1.7.0

  • Valentin Rigal resolved all threads

    resolved all threads

  • Valentin Rigal added 2 commits

    added 2 commits

    Compare with previous version

  • Valentin Rigal changed title from Return a HTTP 400 for too long names using CreateTranscriptionEntities to Return a HTTP 400 for too long names using CreateTranscriptionEntities and CreateEntity

    changed title from Return a HTTP 400 for too long names using CreateTranscriptionEntities to Return a HTTP 400 for too long names using CreateTranscriptionEntities and CreateEntity

  • Valentin Rigal requested review from @erouchet

    requested review from @erouchet

  • Valentin Rigal added 17 commits

    added 17 commits

    Compare with previous version

  • Erwan Rouchet added 6 commits

    added 6 commits

    Compare with previous version

  • Erwan Rouchet enabled an automatic merge when all merge checks for ea1a186a pass

    enabled an automatic merge when all merge checks for ea1a186a pass

  • Please register or sign in to reply
    Loading