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

Revocation contracts - Need help with proxy #76

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

Conversation

vdsilveira
Copy link

@vdsilveira vdsilveira commented Jul 16, 2024

I have added credential revocation smart contracts and tests in Hardhat. The revocation worked when testing the contract directly on the network, but I was not successful using the proxy contract. I need help resolving the issue with the proxy.

@Toktar #54

About the design/specification:
I used the same design as Anoncreds. Basically, the revocation contract checks the credential and creates a credential revocation list, where the revocation status can be changed.

About the proxy:
I used yarn genesis/generate according to the instructions in the README, modified the genesis.json, and launched the network with the new revocation contract included. However, when testing the revocation contract through the proxy, I encountered an error. When testing the contract directly through Remix (as shown in the image below), it worked. Therefore, I believe I need help fixing the proxy error.

TestOnRemix

Signed-off-by: vdsilveira <[email protected]>
Signed-off-by: vdsilveira <[email protected]>
@Toktar
Copy link
Contributor

Toktar commented Aug 13, 2024

Hello @vdsilveira !
Sorry for a delay. Could you share the error message and specification link, you used as a design? Guilty of wanting to see the Revocation delta and accumulators there too much.
If my responses are too slow, please ping me in Hyperledger discord renata.toktar

@vdsilveira vdsilveira marked this pull request as draft September 5, 2024 14:17
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