Skip to content

test: add transfer hook#17

Closed
jean-neiverth wants to merge 2 commits intoupstream-developfrom
jean/hooks-v2-draft
Closed

test: add transfer hook#17
jean-neiverth wants to merge 2 commits intoupstream-developfrom
jean/hooks-v2-draft

Conversation

@jean-neiverth
Copy link
Copy Markdown

Summary

Add a basic transfer hook on hookRegistry. The hook receives an address and one amount as inputs and create a transfer from user to that address.

To Test

  1. Go to /#/8453/swap/hooks
  2. Fill a swap order
  3. Add a transfer post-hook and fill address and amount
  4. The simulation should be successful

Background

This PR will be useful later to compare the before vs. after of proposed changes on Hooks V2 grant proposal.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 2, 2025


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@jean-neiverth jean-neiverth changed the title Jean/hooks v2 draft test: add transfer hook May 2, 2025
@bsvalverde bsvalverde closed this Jan 14, 2026
@bsvalverde bsvalverde deleted the jean/hooks-v2-draft branch January 14, 2026 17:30
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants