search

looxis / laravel-amazon-mws

Simple Amazon MWS Laravel integration
MIT License
31 stars 10 forks source link

GetFeedSubmissionResult Endpoint #15

Closed mirkos93 closed 3 years ago

mirkos93 commented 4 years ago

I added this missing endpoint. it's my first pull request, I hope I haven't done anything wrong.

krizzdev commented 4 years ago

Hey Mirko. Thanks for your contribution! I will check it late afternoon and respond back :)

krizzdev commented 4 years ago

I walked through the code and most parts look pretty good so far. Thank you! What I haven't checked is the response structure. Although the readme.me needs to be updated. Currently I am on vacation until the 17.08. But then I will look at it again and merge it. Have a nice day. 😊

rdfito commented 3 years ago

Hi! I am interested in this PR, any update?

LooxisDev commented 3 years ago

Hi rdfito, currently I'm migrating our apps to laravel 8. I will look into this PR code while making this package laravel 8 conform. I think I will do it tomorrow. Maybe monday.

rdfito commented 3 years ago

Nice! Have a good weekend :)

LooxisDev commented 3 years ago

Added readme and merged it. I will release a version for the current Laravel 7.x before I will make a Laravel 8 release. Have a nice weekend