Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1.89] Post hook (vesting) does not include balance incrementation from a pre-hook #5100

Closed
elena-zh opened this issue Nov 14, 2024 · 3 comments
Labels
Bleu Bug Something isn't working Hooks store Low Severity indicator for defects. It won't cause any major break-down of the system

Comments

@elena-zh
Copy link

  1. Open Hooks tab
  2. Connect to a wallet
  3. Add a pre-hook that will increment a token's balance
  4. Specify a trade with a buy token with incremented balance
  5. Add post-hook: create LLamapay vesting
  6. Pick the last option (manually) and check the balance there

AR: it does not include balance incrementation from a pre-hook
image
image

ER: it should include it.

@elena-zh elena-zh added Bug Something isn't working Low Severity indicator for defects. It won't cause any major break-down of the system Hooks store Bleu labels Nov 14, 2024
@yvesfracari
Copy link
Contributor

Hey Elena! Thanks for reporting this. I think this is solved now, can you test it again? Since this was related only to the hook dapp the changes are on another repo

@yvesfracari
Copy link
Contributor

cc @elena-zh

@elena-zh
Copy link
Author

It works, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bleu Bug Something isn't working Hooks store Low Severity indicator for defects. It won't cause any major break-down of the system
Projects
None yet
Development

No branches or pull requests

2 participants