Skip to content

fix: enable bulk upload form submission (Turbo conflict)#370

Closed
mnietona wants to merge 1 commit into
DocHub-ULB:mainfrom
mnietona:fix/bulk-upload-turbo
Closed

fix: enable bulk upload form submission (Turbo conflict)#370
mnietona wants to merge 1 commit into
DocHub-ULB:mainfrom
mnietona:fix/bulk-upload-turbo

Conversation

@mnietona

Copy link
Copy Markdown
Contributor

Description
This PR fixes the bug where the "Envoyer" button in the bulk upload section was unresponsive, as described in #369.

Fixes #369

Technical Changes

  • Added data-turbo="false" to the bulk upload form in documents/templates/documents/document_upload.html.

Why this fix?
The issue was caused by a conflict with Hotwire Turbo. Turbo was intercepting the form submission but failed to process the standard Django redirect that follows. By disabling Turbo for this specific form, we let the browser handle the POST request and the redirect natively.

Manual Testing

  • Verified that entering a URL and clicking "Envoyer" now correctly redirects the user to the success page (/documents/submit-bulk/<course_slug>).
  • Confirmed that this change doesn't affect other parts of the page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Bulk upload form submission is broken (Turbo conflict)

1 participant