Skip to content

Remove the V1 suffix from the cred def #32

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

Conversation

berendsliedrecht
Copy link
Contributor

@berendsliedrecht berendsliedrecht commented Dec 12, 2022

Dependent upon #23, #27, #30, #31

  • Remove enum wrapper around the cred def
  • Remove V1 suffix after the cred def
  • Removes now useless get_attribute function in the FFI layer

I will leave the revocation stuff for now. @whalelephant Feel free to pick this up (removing the V1 suffix after the structures.)

Work funded by the Government of Ontario.

@berendsliedrecht berendsliedrecht changed the title Update cred def data model Remove the V1 suffix from the cred def Dec 12, 2022
@berendsliedrecht berendsliedrecht force-pushed the update-cred-def-data-model branch from fffe192 to 2e63aa9 Compare December 21, 2022 11:24
@berendsliedrecht berendsliedrecht force-pushed the update-cred-def-data-model branch 5 times, most recently from 0bb86da to 4ca3e4e Compare January 3, 2023 10:30
Work funded by the Government of Ontario.

Signed-off-by: blu3beri <[email protected]>
@berendsliedrecht berendsliedrecht force-pushed the update-cred-def-data-model branch from dee5339 to e5099af Compare January 3, 2023 10:33
@berendsliedrecht berendsliedrecht merged commit c3962f1 into hyperledger:main Jan 3, 2023
@berendsliedrecht berendsliedrecht deleted the update-cred-def-data-model branch January 3, 2023 12:31
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.

2 participants