Skip to content

Conversation

@DaevMithran
Copy link
Contributor

@DaevMithran DaevMithran commented May 28, 2025

This PR fixes the issue in did/update.

The cheqd network expects signatures from all the didUrl in the authentication before and after the update operation.

@DaevMithran DaevMithran requested a review from a team as a code owner May 28, 2025 12:38
@changeset-bot
Copy link

changeset-bot bot commented May 28, 2025

⚠️ No Changeset found

Latest commit: d2d817f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@genaris genaris changed the title fix(cheqd): Include removed authentication signatures on did update fix(cheqd): include removed authentication signatures on did update Aug 5, 2025
@genaris
Copy link
Contributor

genaris commented Aug 5, 2025

Hi @DaevMithran, for some reason I'm unable to re-run the failed check. Can you merge this branch with the latest status of main, so the CI workflow is re-triggered and we can merge this into the next 0.6.0-alphas?

@TimoGlastra
Copy link
Contributor

I have an extended/updated version of this locally, but looking with @DaevMithran as it probably needs some changes to the Cheqd SDK, we can push this in a 0.6.1 release

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