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.

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

Admin change speaker #2984

Open
1 of 2 tasks
ddemaio opened this issue Apr 19, 2022 · 2 comments
Open
1 of 2 tasks

Admin change speaker #2984

ddemaio opened this issue Apr 19, 2022 · 2 comments

Comments

@ddemaio
Copy link

ddemaio commented Apr 19, 2022

I'm submitting a ..

  • Bug Report
  • Feature Request

Current behavior:
From admin, trying to change the speaker is causing an error.

Expected correct behavior:
To add the persons name and allow the admin to change the speaker.

Steps to reproduce:
Go to events, click on event, edit and in the speaker section, add the person who should be the speaker. Then click update proposal

Other information:
Screenshots attached of the area and behavior. This did function correctly last year and I've used it before.

Screenshot from 2022-04-19 20-06-58
Screenshot from 2022-04-19 20-06-20

@hennevogel
Copy link
Member

hennevogel commented Jul 29, 2022

The problem is that the we make an non-idempotent request with the speaker selectize which resets the CSRF token of the form. We would need to add CSRF support to the json/request that search_users_path returns. Some people do this in a response header...

@AndrewKvalheim
Copy link
Member

I tried reproducing this with CSRF protection enabled but haven’t yet hit the error. Can you spot what I’m missing?

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

No branches or pull requests

3 participants