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

feat: upgrade to open attestation v3 #73

Closed
wants to merge 4 commits into from

Conversation

Nebulis
Copy link
Contributor

@Nebulis Nebulis commented Dec 11, 2019

Support for open-attestation v3. Will be merged before #74 but after Open-Attestation/open-attestation#57

Many changes concern typings only. The feature is located in documentToSmartContracts function. I added tests to support v3 open-attestation document.

If you want to try it locally:

  • git clone [email protected]:Open-Attestation/open-attestation.git
  • git checkout feat/add-open-attestation-schema
  • npm link
  • cd /path/to/oa-verify
  • npm link @govtechsg/open-attestation

@Nebulis Nebulis force-pushed the feat/open-attestation-v2 branch from a4842b6 to cc42bce Compare December 18, 2019 06:15
@yehjxraymond yehjxraymond changed the title (wip) fix: upgrade to open attestation v2 (wip) fix: upgrade to open attestation v3 Dec 18, 2019
@Nebulis Nebulis changed the title (wip) fix: upgrade to open attestation v3 feat: upgrade to open attestation v3 Dec 18, 2019
@Nebulis
Copy link
Contributor Author

Nebulis commented Jan 22, 2020

upgraded while implementing verifier manager

@Nebulis Nebulis closed this Jan 22, 2020
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