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

Empty API response ? #55

Closed
nfaugout-lucca opened this issue Oct 13, 2017 · 2 comments
Closed

Empty API response ? #55

nfaugout-lucca opened this issue Oct 13, 2017 · 2 comments
Milestone

Comments

@nfaugout-lucca
Copy link
Contributor

We currently always return the entity when performing a POST or a PUT request.

We already return void for DELETE requests.

When POSTing, after the entity is created, we GET the entity again to be sure that the responded entity correspond a undred % with the one inside the database. It goes the same for PUT.

I wonder if we could just respond 200 OK or 201 Accepted, with the new location of the entity (in case of a POST).

If the JS client wants the new version of the entity, it could issue a new GET request.

This way, we can simplify a lot the code of the AppController and reduce responsibility issue between AppController and RestCollection.

@nfaugout-lucca nfaugout-lucca added this to the RDD v2.1 milestone Oct 13, 2017
@nfaugout-lucca
Copy link
Contributor Author

This is part of a bigger change that consist of issuing more requests from the client, and to fit more with the original REST approach.

@nfaugout-lucca nfaugout-lucca modified the milestones: RDD v2.1, RDD vREST Nov 10, 2017
@nfaugout-lucca
Copy link
Contributor Author

Je ferme cette Issue car j'en parle là : #258

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

1 participant