Changes done in the all the appropriate model files according to the …

OCA / connector-woocommerce

GNU Affero General Public License v3.0

53 stars 106 forks source link

Changes done in the all the appropriate model files according to the … #31

Open techspawn1 opened 8 years ago

techspawn1 commented 8 years ago

…new version of api available in woocommerce version 2.6.4

Fully compatible with woocommerce version(2.4.6) Modified the process of import to get the list of the data Modified the date time format in WOO_DATETIME_FORMAT

pedrobaeza commented 8 years ago

Some comments:

Your changes are not compatible with old versions, so you must create another backend.
Don't replace logo with one with your own company.
Don't add extra lines on files that don't contain any other change.

techspawn commented 8 years ago

@pedrobaeza I have checked with the older versions of woo commerce i.e. 2.4.x and later versions . The code is fully compatible with all the older and newer version of woo commerce. While check the tests conducted by the auto build bot the test is failing because the woocommerce library has not been installed in runbot . So there is no need of creating new backend for our code, just need to replace the code that we have changed.

pedrobaeza commented 7 years ago

Can you check Travis status?

rafaelbn commented 7 years ago

Hi @techspawn1 @techspawn , is it possible to review travis errors and runbot errors in order to get this PR ready for review? Thanks!