#canceled refunds suddenly appear

1 messages ยท Page 1 of 1 (latest)

glass pier
#

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....

inner basin
#

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

glass pier
#

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

inner basin
#

Good to hear it's not causing issues at least!

glass pier
#

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 ๐Ÿ™‚

glass pier
#

@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

inner basin
#

@velvet nest

#

Nice find! I tagged an engineer from our Payments team โ€” he will be back after Christmas to dive into this one ๐Ÿ™‚

glass pier
#

@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

glass pier
#

hi there, any feedback on this @inner basin ?
the bug is still existing

ebon dragon
#

Hey @glass pier, let me check with the team and get back to you ๐Ÿ™Œ

glass pier
#

thanks @ebon dragon

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!

glass pier
#

WOHOOO amazing @ebon dragon thanks a lot ๐Ÿ™‚

ebon dragon
#

Hey @glass pier - it's me again! Just heard back from the team, it's fixed ๐Ÿš€

glass pier
#

haha we'll see...i have cypress on my side ๐Ÿ˜„ , na just kidding, thanks a ton

ebon dragon
#

yes, please let me know if you see some weird behaviour ๐Ÿ˜…

glass pier
#

didnt test manually...but meh....doesnt look good.
ill check it out monday

ebon dragon
#

thanks for heads up @glass pier - looking into this!

glass pier
#

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

ebon dragon
#

aha, I already shared with engineers - but please let me know!