Add QsForm Extractor for Actix

samscott89 / serde_qs

Serde support for querystring-style strings

Apache License 2.0

193 stars 68 forks source link

Add QsForm Extractor for Actix #93

Closed nMessage closed 7 months ago

nMessage commented 11 months ago

Adds QsForm Extractor for Actix-web so that web forms can support arrays and other features of serde_qs

Resolves #52 using code provided by rapodaca

silverpill commented 10 months ago

Very nice, thank you for preparing this pull request.

QsForm works as expected. I have one suggestion though: implement into_inner() directly on QsForm. The requirement to import IntoInner trait feels unnecessary.

samscott89 commented 7 months ago

Thanks for the submission @nMessage

I had a couple of small tweaks:

QsQueryConfig didn't need to be duplicated
Added some tests

Otherwise, it was looking good. Thanks for doing this. Merging this in #98