PAYONE-GmbH / magento-2

PAYONE Payment Extension for Magento 2
28 stars 56 forks source link

#296 fix double invoice #361

Closed piotrmatras closed 4 years ago

piotrmatras commented 4 years ago

Fixes double invoice for appointed notification #296

fjbender commented 4 years ago

Hi @piotrmatras,

Thank you for your contribution!

I'll get this reviewed by one of your developers.

In the meantime, can you maybe take a look why the Test/Unit/Observer/Transactionstatus/AppointedTest.php test breaks with your changes?

Thanks Florian

fjbender commented 4 years ago

Thank you again for your contribution. I will close this PR in favor of #362 where the Unit Tests got mended as well.

We'll move #362 to functional testing and will merge it soon.