search

IntentArchitect / Support

A repository dedicated to handling issues and support queries
3 stars 0 forks source link

Get Request shouldn't have a payload #34

Closed Braydon111 closed 1 year ago

Braydon111 commented 1 year ago

Ask a question

Hi I generated CQRS CRUD and exposed http GET endpoint. It currently requires a payload rather than a query parameter. How do i switch [FromBody] to [FromRoute]?

JonathanLydall commented 1 year ago

Hi @Braydon111,

As HTTP doesn't support payloads for the GET verb, our CRUD script shouldn't be trying to set it up to use the body at all here.

Can you post a screenshot the with the Query node expanded in Intent Architect and also post the generated code for the operation?

Braydon111 commented 1 year ago

Hi Jonathan,

certainly: Screenshot 2023-06-19 161504

Screenshot 2023-06-19 161556

Braydon111 commented 1 year ago

Hi Jonathan,

I manually added the query strings as @joelsteventurner mentioned and it seems to have corrected the issue: image

joelsteventurner commented 1 year ago

Hi @Braydon111

Could I get a screen shot of the SurgeryFormResult entity in your domain ?

Braydon111 commented 1 year ago

Sure: image

joelsteventurner commented 1 year ago

Hi @Braydon111

Thanks for the call to investigate and resolve the issue, that additional FK without an association seems to have been causing issues.

I have logged 2 tickets on our side, one to try prevent this situation in the future and a second to improve how Intent deals with scenarios like this.