What happened to `async_graphql_actix_web::HandlerBuilder`? about async-graphql

[sunli829]

Take a look at this example, add some parameters.

https://github.com/sunli829/async-graphql-examples/blob/6bfcf4498a6134bee144dc770b7c18fde37e45d3/actix-web/upload/src/main.rs#L29

[sunli829]

I didn't think that was flexible enough, so I just removed this.😁
async_graphql_actix_web::GQLRequest is an Extractor that is easier to use.

Look at this
https://github.com/sunli829/async-graphql-examples/blob/fc28d830ba0aaeac3b8ece3ec5e2f725a146aafd/actix-web/starwars/src/main.rs#L7

And this
https://github.com/sunli829/async-graphql-examples/blob/fc28d830ba0aaeac3b8ece3ec5e2f725a146aafd/actix-web/token-from-header/src/main.rs#L20

This should look like an actix_web style.

[sunli829]

I moved all the examples to https://github.com/sunli829/async-graphql-examples/

[sunli829]

The max_file_size and tempdir properties are not supported yet, and this is easy to add because now we have our own multipart parser(https://github.com/sunli829/async-graphql/blob/master/src/http/multipart.rs). I'll add them tomorrow.

[nicolaiunrein]

👍 I was just about to ask. I liked the builder though and I think it made it very intuitive for people getting started with the library.

[sunli829]

Builder is convenient, but not easy to combine with other Extractor, I haven't found a good way yet.

[nicolaiunrein]

I agree. And I think we should get the features straight first and can add convenience functionality once everything has settled.