mecachisenros/woocommerce_civicrm

Plugin refactoring

mecachisenros opened this issue · 5 comments

mecachisenros commented

This is not an issue but rather a question.

I've recently run into a small issue with the latest WC 3.0+ where the order properties cannot be accessed directly anymore (protected properties) and we have to use getter methods like get_billing_email() so I've gave it a go and refactored the plugin aiming for, hopefully, get it ready for publishing in the WordPress plugin directory.

The question: would you @veda-consulting be interested in publishing it?

I've created a v2.0 branch, I can prepare a PR and I'll be happy to help improving the integration.

mecachisenros commented

@veda-consulting I've made a few more improvements, support for State/Province, added a few hooks and an option to sync woocommerce/civicrm mapped addresses both ways #4

veda-consulting commented

@mecachisenros thanks for those, if you can prepare a PR we would be happy to release version 2.0

mecachisenros commented

@veda-consulting perfect, I just need to push a couple more fixes and then I'll prepare the PR, ETA tomorrow evening

mecachisenros commented

@veda-consulting as promised, PR created, feel free to review/ask clarifications

veda-consulting commented