mollie / magento2

Mollie Payments for Magento 2
https://www.mollie.com
Other
100 stars 53 forks source link

New Payment IDs breaks the module #670

Closed wouter-toppy closed 1 year ago

wouter-toppy commented 1 year ago

Thank you for creating this pull request! To make the best use of your and our time we created this checklist to get the best possible pull requests:

This PR touches code in the following areas (Check what is applicable):

Frontend

Backend

Order Processing (Mollie communication)

Please describe the bug/feature/etc this PR contains: Fixes: #669

wouter-toppy commented 1 year ago

@Frank-Magmodules

mollierobbert commented 1 year ago

This will work for now, but the numeric prefix will increase to more than a single digit as the dataset grows further. (A few years down the road.)

Sorry, I see this MR was just a hot fix. Thanks for the support!

The fix that Magmodules released now is compatible with multi-digit prefixes as well.