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

introduce wallet connect #713

Merged
merged 152 commits into from
Aug 13, 2024
Merged

introduce wallet connect #713

merged 152 commits into from
Aug 13, 2024

Conversation

shamilovtim
Copy link
Member

@shamilovtim shamilovtim commented Jun 29, 2024

A minimal implementation for connecting a wallet through an intermediary server using an OIDC protocol plus bespoke additions. There are improvements that can be made and future feature requests: #794 #749 #748 etc.

Copy link

changeset-bot bot commented Jun 29, 2024

🦋 Changeset detected

Latest commit: 19bd7b6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages
Name Type
@web5/agent Minor
@web5/api Patch
@web5/credentials Patch
@web5/crypto Patch
@web5/crypto-aws-kms Patch
@web5/dids Patch
@web5/identity-agent Patch
@web5/proxy-agent Patch
@web5/user-agent Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

github-actions bot commented Jun 29, 2024

TBDocs Report

🛑 Errors: 0
⚠️ Warnings: 7

@web5/api

  • Project entry file: packages/api/src/index.ts

@web5/crypto

  • Project entry file: packages/crypto/src/index.ts
📄 File: ./packages/crypto/src/utils.ts
⚠️ extractor:typedoc:missing-docs: utils.CryptoUtils.__type.randomPin (Property) does not have any documentation.
⚠️ extractor:typedoc:missing-docs: utils.CryptoUtils.__type.randomUuid (Property) does not have any documentation.
⚠️ extractor:typedoc:missing-docs: utils.CryptoUtils.__type.randomBytes (Property) does not have any documentation.
⚠️ extractor:typedoc:missing-docs: utils.CryptoUtils.__type.isWebCryptoSupported (Property) does not have any documentation.
⚠️ extractor:typedoc:missing-docs: utils.CryptoUtils.__type.getJoseSignatureAlgorithmFromPublicKey (Property) does not have any documentation.
⚠️ extractor:typedoc:missing-docs: utils.CryptoUtils.__type.checkValidProperty (Property) does not have any documentation.
⚠️ extractor:typedoc:missing-docs: utils.CryptoUtils.__type.checkRequiredProperty (Property) does not have any documentation.

@web5/crypto-aws-kms

  • Project entry file: packages/crypto-aws-kms/src/index.ts

@web5/dids

  • Project entry file: packages/dids/src/index.ts

@web5/credentials

  • Project entry file: packages/credentials/src/index.ts

TBDocs Report Updated at 2024-08-13T18:46:01Z 19bd7b6

thehenrytsai
thehenrytsai previously approved these changes Aug 13, 2024
Copy link
Member

@thehenrytsai thehenrytsai left a comment

Choose a reason for hiding this comment

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

🚀 🎖️ 🎊 🥳 🎆 🍾

Optional consideration to merge #837 as I doubt #830 itself will get rid of tbdocs errors.

* cleanup crypto utils

* changeset

* Update index.ts

* finish: delete package.json utils export
thehenrytsai
thehenrytsai previously approved these changes Aug 13, 2024
thehenrytsai
thehenrytsai previously approved these changes Aug 13, 2024
Copy link
Member

@thehenrytsai thehenrytsai left a comment

Choose a reason for hiding this comment

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

Third time lucky.

Copy link
Member

@thehenrytsai thehenrytsai left a comment

Choose a reason for hiding this comment

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

4th time is the charm.

@shamilovtim shamilovtim merged commit 5ed8798 into main Aug 13, 2024
35 checks passed
@shamilovtim shamilovtim deleted the tshamilov/connect-v2 branch August 13, 2024 19:13
This was referenced Aug 14, 2024
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.

4 participants