Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: stringify for normalization json-pointer for JSON string data #10

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

namhoang1604
Copy link

When createProof using the normalization json-pointer with the document content the value is JSON string, that will raise an error.
Example document:

{
  "@context": [
    "https://www.w3.org/2018/credentials/v1",
    "https://www.w3.org/2018/credentials/examples/v1"
  ],
  "type": [
    "VerifiableCredential",
    "UniversityDegreeCredential"
  ],
  "credentialSubject": {
    "name": "Jane Smith",
    "degree": {
      "type": "BachelorDegree",
      "name": "Bachelor of Science and Arts",
      "degreeSchool": "{\"name\": \"Example University\"}"
    }
  },
  "issuer": {
    "id": "did:web:example.com"
  }
}

@namhoang1604 namhoang1604 force-pushed the fix/stringify-issue-normalization branch 7 times, most recently from 755abe8 to 07fa67a Compare November 23, 2023 07:56
@namhoang1604 namhoang1604 force-pushed the fix/stringify-issue-normalization branch from 07fa67a to cd711b9 Compare November 23, 2023 08:30
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.

1 participant