reverbdotcom / reverb-magento

Magento 1.x plugin for syncing with Reverb
Other
7 stars 10 forks source link

Category UUID Support #229

Closed dunagan5887 closed 8 years ago

dunagan5887 commented 8 years ago

Fix #223

skwp commented 8 years ago

Can we get rid of the CSV stuff now?

dunagan5887 commented 8 years ago

The CSV stuff is being used by the initial migration which updates the Reverb category list. Are you saying you want to modify that migration script to use the new JSON API functionality?

skwp commented 8 years ago

Yes the JSON api should be the source of truth for everything

dunagan5887 commented 8 years ago

The migration script is now based off of the API JSON data response. I removed all architecture related to the CSV file parsing. In terms of clients, I would recommend that they perform this update in maintenance mode during off hours just to be safe. I wrapped everything in try-catch blocks, but who knows what third-party extensions people have loaded on their instances