The proccess view should use form validation
Closed this issue · 5 comments
Right now we don't do any validation on the process endpoint. A massive ick as it does not feel like an adequate user/external input validation.
wagtail-ai/src/wagtail_ai/views.py
Lines 77 to 94 in 13e479c
We want to replace the HttpRequest.POST
references and replace it with a Django form validation.
We should wrap this into a Django form to benefit from any Django defaults and put good practice in place for the future.
Hi @kituuu - thanks for offering to contribute!
Feel free to work on this one if you'd like to.
Hey, @tomusher, can you tell me where I can find contribution instructions and dev environment setup instructions?
Current contribution.md is empty... (https://github.com/wagtail/wagtail-ai/blob/main/CONTRIBUTING.md)