- Issue created by @thomas wijnands
- Merge request !11[3505883] feat: improve onNotify logic and add support for multiple method calls → (Open) created by Unnamed author
- 🇳🇱Netherlands ndf Amsterdam
Hi Thomas, thank for the patch.
On first sight it looks good to me.
I noticed that the automated tests don't run. Just now I created 📌 Commerce Mollie GitLab CI for automated tests Active to fix this.
The automated test suite has test-coverage for the payments states handling that is changes in this ticket. So we for merging the test-suite is a blocker. - 🇳🇱Netherlands thomas wijnands
Hi Niels,
Thanks! I'll wait for the pipelines to have ran then.
Best regards,
Thomas Wijnands - 🇳🇱Netherlands thomas wijnands
just updated my patch since i thought MolliePaymentStatus was passed down as a enum class instead of a plain string.