Include handler for `payment_pointer.web_monetization` webhook event in Mock ASE

interledger / rafiki

An open-source, comprehensive Interledger service for wallet providers, enabling them to provide Interledger functionality to their users.

https://rafiki.dev/

Apache License 2.0

234 stars 84 forks source link

Include handler for `payment_pointer.web_monetization` webhook event in Mock ASE #2057

Closed sabineschaller closed 4 months ago

sabineschaller commented 11 months ago

Context

The Mock ASE currently does not handle payment_pointer.web_monetization webhook events. Since it should also serve as a reference for actual integrators, all webhook events should be handled.

Note: As of the creation of this issue, it is still called payment_pointer. However, we are in the process of changing this to wallet_address.

Todos

[x] include handler here
[x] handle event case here

manulpatel commented 11 months ago

Hello @sabineschaller! I would like to work on this issue. Could you please assign me?

sabineschaller commented 11 months ago

Hi @manulpatel , it's yours 🙂

Shubham2503 commented 11 months ago

Hey team, @sabineschaller @manulpatel, Can I take this one, if this issue is still open?

BlairCurrey commented 10 months ago

Currently the docs indicate calling the WithdrawEventLiquidity endpoint in this webhook handler. However, this handler is being deprecated and we should be able to use createWalletAddressWithdrawal instead.

Also, this webhook never actually fires. We might be able to manually test by:

changing walletAddressService.create to do something like this: https://github.com/interledger/rafiki/commit/70a03ad05bb2798f42b2b24cde654848db348b6c
create a new wallet address via postman
call triggerWalletAddressEvents via postman which should trigger handleWalletAddressWebMonetization