Skip to content

implemented the transfer#834

Open
nafiuishaaq wants to merge 1 commit into
DistinctCodes:mainfrom
nafiuishaaq:feat/transfer
Open

implemented the transfer#834
nafiuishaaq wants to merge 1 commit into
DistinctCodes:mainfrom
nafiuishaaq:feat/transfer

Conversation

@nafiuishaaq
Copy link
Copy Markdown
Contributor

Completed

Implemented two-step admin transfer support in the assetsup contract.

Changes made

  • Added admin_transfer.rs

    • propose_admin_transfer(env, new_admin)
    • accept_admin_transfer(env)
    • cancel_admin_transfer(env)
    • get_pending_admin(env)
    • Emits admin_transfer_proposed and admin_transfer_completed
  • Updated lib.rs

    • Added DataKey::PendingAdmin
    • Registered pub(crate) mod admin_transfer;
  • Extended admin.rs

    • Happy path: propose + accept
    • Non-admin propose fails
    • Wrong address accept fails
    • Cancel and re-propose behavior

Note

I implemented this in admin_transfer.rs because this workspace contains the assetsup contract rather than a contracts/opsce folder.

Made changes.

Closes #819
Closes #812

@vercel
Copy link
Copy Markdown

vercel Bot commented May 30, 2026

@nafiuishaaq is attempting to deploy a commit to the naijabuz's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 30, 2026

@nafiuishaaq Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant