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.

Sign up for GitHub

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

Jump to bottom

Questionnaire reordering fails #2217

Closed
janno42 opened this issue Jun 9, 2024 · 1 comment · Fixed by #2222
Closed

Questionnaire reordering fails #2217

janno42 opened this issue Jun 9, 2024 · 1 comment · Fixed by #2222
Labels
[C] Backend Focuses on backend implementation [P] Major Major priority [S] Small This issue should require only small changes. [T] Bug This is a bug. We don't like it. Please get rid of it.
Milestone
Summer 2024

Comments

@janno42
Copy link
Member

janno42 commented Jun 9, 2024 •
edited
Loading

When reordering questionnaires on the questionnaire index page, a 403 error is returned. This happens because the CSRF token is missing: Forbidden (CSRF token missing.): /staff/questionnaire/update_indices
@janno42 janno42 added [C] Backend Focuses on backend implementation [T] Bug This is a bug. We don't like it. Please get rid of it. [P] Major Major priority [S] Small This issue should require only small changes. labels Jun 9, 2024
@janno42 janno42 added this to the Summer 2024 milestone Jun 10, 2024
@janno42
Copy link
Member Author

janno42 commented Jun 10, 2024

The X-CSRFToken in the request header was removed in 5b1cf1c (part of #2013). Turns out this code is actually still needed for Sortable in the QuestionnaireGrid :)

@Kakadus Kakadus mentioned this issue Jun 10, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
[C] Backend Focuses on backend implementation [P] Major Major priority [S] Small This issue should require only small changes. [T] Bug This is a bug. We don't like it. Please get rid of it.
Milestone
Summer 2024
Development

Successfully merging a pull request may close this issue.

replace last jquery with plain browser api Kakadus/EvaP
1 participant
@janno42