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

Feature/336 upgrade to veramo 2 #396

Merged
merged 65 commits into from
Aug 18, 2024

Conversation

theblockstalk
Copy link
Contributor

@theblockstalk theblockstalk commented Jun 18, 2024

  1. upgraded ALL packages to latest
  2. Jest runs all tests using ESM under
    • typeorm, uuid and ws packages could not be bundled fully with ESM so exceptions were made in jest.config.ts
    • yarn run cli also using ESM bundles
  3. Added new veramo.ts and unit tests. to ensure we are ready for next tasks that depended on this upgrade Upgrade to use Veramo library #337 to be ready for Upgrade to use latest did-resolver, did-jwt and did-jwt-vc libraries Tonomy-ID#758
  4. removed did-jwk.ts and upgraded to use did-key.ts instead. this solves Reduce URL down to below recommended sizes #316
  5. removed the API and utils builds and conditional exports
  6. removed unused base64.ts helper functions and unit tests
  7. added new CI pipeline to check for latest packages - but it does not block PR merges, just there as a warning to remind developers
  8. upgraded eslint to use the new configuration file and conventions. Also removing and updating config dependency packages
  9. upgraded to yarn 4.3.1
  10. installed debug for debug logging
  11. fixed issue with ExternalUser.getIssuer() using the wrong keyManager
  12. fixed all tests to work again

debug(`Credential verified`, result.verified);
}

export async function veramo2() {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

veramo and veramo2 name.. may be we can update this name?

@sadiabbasi sadiabbasi merged commit bbd4b1a into development Aug 18, 2024
3 of 4 checks passed
@theblockstalk
Copy link
Contributor Author

🎉 This PR is included in version 0.21.0-development.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

@theblockstalk
Copy link
Contributor Author

🎉 This issue has been resolved in version 0.22.0-development.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@theblockstalk
Copy link
Contributor Author

🎉 This issue has been resolved in version 0.22.0-rc.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@theblockstalk
Copy link
Contributor Author

🎉 This issue has been resolved in version 0.22.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants