hi there. we have a refund managing system since a long time in our integration. its possible to cancel pending refunds. until a while ago, those canceled refunds where just deleted, from our integration (that fetches mollie live data) and the mollie dashboard.
but recently it feels as if mollie introduced a new "canceled" status, because they appeared again (also in the mollie dashboard), but i think not all the time.
so has there been anything changed in the api? maybe we just need to filter out canceled refunds....
#canceled refunds suddenly appear
1 messages ยท Page 1 of 1 (latest)
Indeed we have been working on making deleted refunds visible. We have in recent years considered it a flaw in our early API design to just make them disappear, which led to some annoying edge cases.
I'm not sure though what the communication strategy has been around this change... ๐ฌ Let me ping some people from the appropriate team
hi
aaaa....ok perfect, its not a big deal, i think we just filter it out when fetching it, but nope, it hasn't been communicated
Good to hear it's not causing issues at least!
yeah...at least i think it won't if i remember it correctly how i've built it haha ๐
but we can do anything, so no worries ๐
@inner basin we found a bug in the Mollie API. you accidentally still include the canceled refunds in the line items of the order (already refunded count)
steps to reproduce
- create an order
- do a full refund with line item information
- mollie order line items shows 1 is refunded
- now cancel the refund (HTTP DELETE)
- the order still shows 1 refunded count
problem and impact -> another refund isnt possible because it says "already refunded"
hope this helps
@velvet nest
Nice find! I tagged an engineer from our Payments team โ he will be back after Christmas to dive into this one ๐
@inner basin always happy to help. we'll probably cover it with a cypress test that is now red, and are happy once its green again ๐
but now..............christmas vacation
take care, merry christmas
hi there, any feedback on this @inner basin ?
the bug is still existing
Hey @glass pier, let me check with the team and get back to you ๐
thanks @ebon dragon
Hey @glass pier! I talked to the payments team, they confirmed the issue and added it to the backlog. Once it's fixed, I will follow up once again with you! ๐ Appreciate the heads up as always!
WOHOOO amazing @ebon dragon thanks a lot ๐
Hey @glass pier - it's me again! Just heard back from the team, it's fixed ๐
haha we'll see...i have cypress on my side ๐ , na just kidding, thanks a ton
yes, please let me know if you see some weird behaviour ๐
didnt test manually...but meh....doesnt look good.
ill check it out monday
thanks for heads up @glass pier - looking into this!
yep, but let me firs tdouble check, weve improved the cypress test for this use case, maybe something is wrong in here too
background story -> we updated it again 2 days ago with a new scenario for that bug in mollie
aha, I already shared with engineers - but please let me know!