Skip to content

feat: wallet creation with DCP protocol#205

Open
thackerronak wants to merge 15 commits intoSAP:mainfrom
thackerronak:feat/enable_dcp_flow
Open

feat: wallet creation with DCP protocol#205
thackerronak wants to merge 15 commits intoSAP:mainfrom
thackerronak:feat/enable_dcp_flow

Conversation

@thackerronak
Copy link
Copy Markdown
Contributor

Description

Wallet creation with DCP protocol

Why

Wallet support DCP issuance flow

Issue

1160

Checklist

Please delete options that are not relevant.

  • I have performed a self-review of my own code
  • I have successfully tested my changes locally
  • I have added tests that prove my changes work
  • I have checked that new and existing tests pass locally with my changes
  • I have commented my code, particularly in hard-to-understand areas
  • I have added copyright and license headers, footers (for .md files) or files (for images)

@thackerronak
Copy link
Copy Markdown
Contributor Author

Hey @Phil91, this ticket is open for review.

Comment thread src/processes/DimProcess.Library/DimProcessHandler.cs Outdated
@Siegfriedk
Copy link
Copy Markdown
Member

Siegfriedk commented Jul 30, 2025

@thackerronak i can't do a proper review as i'm not familiar enough with the code. But please check my comment and also do an dependency update (see the other PRs from Dependabot).

Is there anyone else you are aware of who can do a proper review? I can merge it after

I would also suspect a release update like in this git commit: 48867fe

@thackerronak
Copy link
Copy Markdown
Contributor Author

@thackerronak i can't do a proper review as i'm not familiar enough with the code. But please check my comment and also do an dependency update (see the other PRs from Dependabot).

Is there anyone else you are aware of who can do a proper review? I can merge it after

I would also suspect a release update like in this git commit: 48867fe

  1. Dependabot has undergone a significant version update, necessitating code changes to accommodate this new task instead of the current one
  2. @leandro-cavalcante can help here for review
  3. I checked the released update commit link and found that it will be created after all the features are merged.

@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented Oct 6, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ thackerronak
❌ ShahinRashidbayli
You have signed the CLA already but the status is still pending? Let us recheck it.

@thackerronak
Copy link
Copy Markdown
Contributor Author

@Phil91 available for review.

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