Skip to content

refactor: update DID creation response#236

Merged
bhavanakarwade merged 5 commits intodevelopfrom
refactor/did-create-response-for-polygon
Feb 21, 2025
Merged

refactor: update DID creation response#236
bhavanakarwade merged 5 commits intodevelopfrom
refactor/did-create-response-for-polygon

Conversation

@bhavanakarwade
Copy link
Copy Markdown
Contributor

@bhavanakarwade bhavanakarwade commented Feb 21, 2025

What ?

  • Modified the response format for DID creation on the Polygon ledger.
  • Made the keyType property optional in the DID creation functionality, as it is not mandatory for Polygon.

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>
Comment thread src/controllers/did/DidController.ts Outdated
Copy link
Copy Markdown

@prasadgkalamkar prasadgkalamkar left a comment

Choose a reason for hiding this comment

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

Define the type of the variable.

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>
Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>
Copy link
Copy Markdown
Contributor

@tipusinghaw tipusinghaw left a comment

Choose a reason for hiding this comment

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

LGTM

@sonarqubecloud
Copy link
Copy Markdown

@bhavanakarwade bhavanakarwade merged commit 3268718 into develop Feb 21, 2025
pranalidhanavade pushed a commit that referenced this pull request Mar 6, 2025
* refactor: create did response

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>

* fix: resolved comments

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>

* fix: handle validations for polygon

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>

---------

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>
KambleSahil3 pushed a commit that referenced this pull request Apr 30, 2025
* refactor: create did response

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>

* fix: resolved comments

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>

* fix: handle validations for polygon

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>

---------

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>
KambleSahil3 pushed a commit that referenced this pull request Apr 30, 2025
* refactor: create did response

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>

* fix: resolved comments

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>

* fix: handle validations for polygon

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>

---------

Signed-off-by: bhavanakarwade <bhavana.karwade@ayanworks.com>
Signed-off-by: KambleSahil3 <sahil.kamble@ayanworks.com>
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.

4 participants