Skip to content

DCKA 5575 web wallet delegation support - #570

Merged
maycon-mello merged 3 commits into
masterfrom
DCKA-5575-web-wallet-delegation-support
Jul 29, 2026
Merged

DCKA 5575 web wallet delegation support#570
maycon-mello merged 3 commits into
masterfrom
DCKA-5575-web-wallet-delegation-support

Conversation

@maycon-mello

Copy link
Copy Markdown
Collaborator
  • Add delegation support to web wallet

@maycon-mello
maycon-mello force-pushed the DCKA-5575-web-wallet-delegation-support branch from 2bd5396 to d07607c Compare July 29, 2026 16:54
@maycon-mello
maycon-mello requested a review from Copilot July 29, 2026 16:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds delegation support to the web wallet SDK by exposing core delegation utilities and providing wallet-bound helpers to participate in delegation DIDComm flows.

Changes:

  • Re-export delegation-related APIs (offers, issuance, policy/chain lookup, revocation) from @docknetwork/wallet-sdk-web.
  • Add wallet instance helpers: enableDelegation(), createDelegationOffer(), acceptDelegationOffer(), getDelegationDetails(), and handleMessage().
  • Add Jest coverage for the new exports and wallet-bound delegation behavior.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
packages/web/src/index.js Imports and re-exports delegation APIs; adds delegation helpers on the initialized wallet instance.
packages/web/src/index.test.js Extends mocks and adds tests for delegation re-exports and wallet-bound delegation helpers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/web/src/index.js Outdated
Comment on lines +693 to +695
createDelegationOffer: params =>
createDelegationOffer({wallet, ...params}),

Comment thread packages/web/src/index.js Outdated
Comment thread packages/web/src/index.js Outdated

@mike-parkhill mike-parkhill left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment on a comment and there are some linting issues, but otherwise lgtm

@maycon-mello
maycon-mello merged commit a3f6667 into master Jul 29, 2026
7 checks passed
@maycon-mello
maycon-mello deleted the DCKA-5575-web-wallet-delegation-support branch July 29, 2026 18:52
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.

3 participants