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

CLI / ATA Delegation #201

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

murmky
Copy link

@murmky murmky commented Feb 3, 2025

Adds two commands to the CLI:

  • delegate_token_account, which delegates a token account owned by a vault to a designated third-party key
  • delegated_token_transfer, which enables the new key to withdraw the funds from the specified token account where the delegation was made

Requires #196 to be merged first (branch is rebased on ck/more-cli).

Copy link

codecov bot commented Feb 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.70%. Comparing base (c736bca) to head (ca356be).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #201   +/-   ##
=======================================
  Coverage   80.70%   80.70%           
=======================================
  Files          91       91           
  Lines        9944     9944           
=======================================
  Hits         8025     8025           
  Misses       1919     1919           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@coachchucksol
Copy link
Contributor

Hey @murmky! Thanks for the PR!

Could you kindly merge in master and then run ./format in the root directory and fix any errors? This will allow our CI/CD to pass!

Cheers!

@murmky
Copy link
Author

murmky commented Feb 20, 2025

@coachchucksol should be better now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants