-
Notifications
You must be signed in to change notification settings - Fork 28
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
feat: Store VC and retrieve VC using edge client #26
Conversation
create the profile and get the prefix of the ID from the profile issue-47This comment was generated by todo based on a
|
b2976c7
to
a0e6ca5
Compare
Codecov Report
@@ Coverage Diff @@
## master #26 +/- ##
==========================================
+ Coverage 89.6% 90.24% +0.63%
==========================================
Files 5 5
Lines 202 246 +44
==========================================
+ Hits 181 222 +41
- Misses 11 13 +2
- Partials 10 11 +1
Continue to review full report at Codecov.
|
// TODO create the profile and get the prefix of the ID from the profile issue-47 | ||
id = "https://example.com/credentials/1872" | ||
// ID is the identifier for the verifiable credential | ||
ID = "https://example.com/credentials/1872" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is not supposed to be exported, it will be removed anyway
// ID is the identifier for the verifiable credential | ||
ID = "https://example.com/credentials/1872" | ||
// VaultID TODO create the profile and get the prefix of the ID from the profile issue-47 | ||
VaultID = "Credentials" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why exported? You can add it to profile after rebase.
} | ||
svc.registerHandler() | ||
|
||
_, err = svc.client.CreateDataVault(&operation.DataVaultConfiguration{ReferenceID: VaultID}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be done during profile creation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@troyronda can you please suggest if its okay to create the data vault during the creation of issuer profile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have thought so.
bc615c1
to
41b2260
Compare
d340045
to
33c159d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We may want to configure vault in the profile later on.
remove the stripping of the string issue-34This comment was generated by todo based on a
|
remove the stripping of the string issue-34This comment was generated by todo based on a
|
closes: trustbloc#9 Signed-off-by: talwinder.kaur <talwinder.kaur@securekey.com>
closes: #9