Skip to content
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

2.0.0 update #19

Merged
merged 306 commits into from
Nov 22, 2020
Merged

2.0.0 update #19

merged 306 commits into from
Nov 22, 2020

Conversation

dingo-d
Copy link
Owner

@dingo-d dingo-d commented Mar 31, 2020

Plugin rewrite.

To do:

  • Explore the implementation of interfaces in the plugin (with the autoloader).
  • Add wp-browser test suite
  • Modify the expiry date of the offers/invoices (rok_placanja)
  • Pull the translation of customer note based on the language selected
  • Issue: if 2 transactions are marked as complete, only one will send the call to solo API for completion.
  • Issue: When changing the order from on hold to completed in the order screen the API call is made again (support link)
  • Link the invoice from the Solo service with the WooCommerce invoice (support link)
  • Make company field mandatory if R1 type of invoice is selected (support link)
  • Check if the offer/bill is created twice (support link)
  • Check the possible rounding errors (support link)
  • API call made twice (using Corvuspay integration) (support link)
  • Check the Braintree integration

Changed:

  • Entire plugin structure (PSR-12, PSR-4 autoloading, DI container)
  • Added automated tests
  • Fixed outstanding issues
  • Update settings screen

@dingo-d dingo-d added this to the 2.0.0 milestone Mar 31, 2020
@dingo-d dingo-d self-assigned this Mar 31, 2020
@dingo-d dingo-d changed the title [WIP] 2.0.0 update 2.0.0 update Nov 21, 2020
@dingo-d dingo-d merged commit 711448c into develop Nov 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

To do for the next releases
2 participants