Skip to content

Reviews API #4486

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

Closed
wants to merge 4 commits into from
Closed

Reviews API #4486

wants to merge 4 commits into from

Conversation

TsPG128
Copy link

@TsPG128 TsPG128 commented May 12, 2016

I have extended the API with reviews. Current calls supported:

GET /V1/reviews (method getList)
GET /V1/reviews/:reviewId (method get)
POST /V1/reviews (method save)
DELETE /V1/reviews/:reviewId (method delete)

For it, I created a ReviewsRepository, along with interfaces for Review and ReviewSearchResult.
The GET /V1/reviews call supports searchCriteria.
I have put the restrictions to the 'Magento_Review::reviews_all' privileges on all of the calls.

I guess a bit of cleanup in the ReviewInterface itself should be still done, for what data should be returned and posted.

Additionally, I've created a setter function for status, that is setting the status id in the review based on the status code, along with a getStatus function, that is returning the status code based on the status id in the review.

Parts that are in the same module but not done are: ratings, summary, status calls.
Tests are not done yet too for it.

@ilol
Copy link

ilol commented Aug 4, 2016

Internal ticket: MAGETWO-56385

@vkorotun vkorotun added linked improvement Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development and removed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development labels Aug 4, 2016
@vkorotun vkorotun added Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development and removed linked labels Aug 22, 2016
@okorshenko okorshenko self-assigned this Apr 20, 2017
@okorshenko okorshenko added this to the April 2017 milestone Apr 20, 2017
@okorshenko
Copy link
Contributor

okorshenko commented Apr 20, 2017

Hi @thomas-spigel
Thank you for your Pull Request. Sorry for delay with response. Are still interested in implementation of review WebAPI? We have created design for this functionality so we can share it with you to finalize this PR.

@okorshenko
Copy link
Contributor

okorshenko commented May 9, 2017

Hi @thomas-spigel
Here is proposed Review API implantation design. #9570 If you are interested in this implementation, please, let us know and we will be working with you closely on this implementation.

Closing this PR for now. But this PR is a good start for Review API implementation. 👍

@okorshenko okorshenko closed this May 9, 2017
magento-engcom-team pushed a commit that referenced this pull request Jul 18, 2019
[Owls] Page Builder Render Update & Skip MFTF Tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Customer improvement Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Progress: needs update Progress: reject
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants