Skip to content

feat: intergration of SafePal wallet#1450

Merged
Adamj1232 merged 3 commits intothirdweb-dev:v2-web3-onboard-developfrom
QiAbc:v2-web3-onboard-develop
Jan 26, 2023
Merged

feat: intergration of SafePal wallet#1450
Adamj1232 merged 3 commits intothirdweb-dev:v2-web3-onboard-developfrom
QiAbc:v2-web3-onboard-develop

Conversation

@QiAbc
Copy link
Copy Markdown
Contributor

@QiAbc QiAbc commented Jan 9, 2023

Description

Checklist

  • The version field in package.json is incremented following semantic versioning
  • The box that allows repo maintainers to update this PR is checked
  • I tested locally to make sure this feature/fix works
  • I have run yarn file-check, yarn type-check & yarn build to confirm there are not any associated errors
  • This PR passes the Circle CI checks

If this PR includes changes to add an injected wallet or SDK wallet module:

Please complete the following using the internal demo package.
To run this demo use the command yarn && yarn dev to get the project running at http://localhost:8080/

Tests with demo app (injected)

  • send transaction
  • switch chains
  • sign message
  • sign typed message
  • disconnect

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 9, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
web3-onboard-docs ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 26, 2023 at 7:15PM (UTC)

@Adamj1232
Copy link
Copy Markdown
Contributor

@QiAbc thank you for the PR! We will get it review shortly.
Can you please complete the wallet integration checklist and confirm full functionality of SafePal wallet with W3O?
Thanks again!

@Adamj1232
Copy link
Copy Markdown
Contributor

@QiAbc friendly ping on pulling the latest v2-web3-onboard-develop branch and completing the new wallet checklist within the description 😃

@Adamj1232
Copy link
Copy Markdown
Contributor

@QiAbc friendly ping 😄

Copy link
Copy Markdown
Contributor

@Adamj1232 Adamj1232 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR and support!

@Adamj1232 Adamj1232 merged commit 130c643 into thirdweb-dev:v2-web3-onboard-develop Jan 26, 2023
Adamj1232 added a commit that referenced this pull request Jan 30, 2023
* feat: intergration of SafePal wallet (#1450)

* feat: intergration of SafePal wallet

* cleanup

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>

* Update sdk version for TP (#1495)

* Trust wallet integration & SDK (#1463)

* feat: create trust wallet SDK

* docs: add Trust wallet SDK docs link

* docs: add trust SDK option

* docs:  add trust SDK option

* chore: add trust SDK jobs

* feat: add trust as part of the default wallets

* feat: add trust type

* fix: not identifying injected trust wallet

* fix: building error

* fix: only verify if trustwallet exists

* fix: wallets conflicting with each other

* chore: increment version

* chore: revert version

* docs: update documentation with Trust example

* docs: update description

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>

* chore: update version

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>

* docs: resolve typo

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>

* fix: incorrectly checking providers

* Add trust to docs site

* Add docs and bump version

* Fix deploy config

---------

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>

* Bump cookiejar from 2.1.3 to 2.1.4 (#1482)

Bumps [cookiejar](https://github.com/bmeck/node-cookiejar) from 2.1.3 to 2.1.4.
- [Release notes](https://github.com/bmeck/node-cookiejar/releases)
- [Commits](https://github.com/bmeck/node-cookiejar/commits)

---
updated-dependencies:
- dependency-name: cookiejar
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump cookiejar from 2.1.3 to 2.1.4 in /examples/with-vite-react (#1483)

Bumps [cookiejar](https://github.com/bmeck/node-cookiejar) from 2.1.3 to 2.1.4.
- [Release notes](https://github.com/bmeck/node-cookiejar/releases)
- [Commits](https://github.com/bmeck/node-cookiejar/commits)

---
updated-dependencies:
- dependency-name: cookiejar
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump cookiejar from 2.1.3 to 2.1.4 in /packages/demo (#1485)

Bumps [cookiejar](https://github.com/bmeck/node-cookiejar) from 2.1.3 to 2.1.4.
- [Release notes](https://github.com/bmeck/node-cookiejar/releases)
- [Commits](https://github.com/bmeck/node-cookiejar/commits)

---
updated-dependencies:
- dependency-name: cookiejar
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump cookiejar from 2.1.3 to 2.1.4 in /docs (#1486)

Bumps [cookiejar](https://github.com/bmeck/node-cookiejar) from 2.1.3 to 2.1.4.
- [Release notes](https://github.com/bmeck/node-cookiejar/releases)
- [Commits](https://github.com/bmeck/node-cookiejar/commits)

---
updated-dependencies:
- dependency-name: cookiejar
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump cookiejar from 2.1.3 to 2.1.4 in /examples/with-nextjs-13 (#1484)

Bumps [cookiejar](https://github.com/bmeck/node-cookiejar) from 2.1.3 to 2.1.4.
- [Release notes](https://github.com/bmeck/node-cookiejar/releases)
- [Commits](https://github.com/bmeck/node-cookiejar/commits)

---
updated-dependencies:
- dependency-name: cookiejar
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Feat/add okx wallet (#1469)

* feature: add okx wallet to injected

* feature: add okx wallet to injected

* feature: add okx wallet to injected

* feature: add okx wallet to injected

* feature: add okx wallet to injected

* Update okx wallet usage

* Add debugging code for okx team

* Update svg for wallet

* Clean up logs

---------

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>

* Update versions for release

* Address a couple of warnings from type-check

* Bump sdk version in core

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: QiAbc <121917904+QiAbc@users.noreply.github.com>
Co-authored-by: karim-yassine <118207693+karim-yassine@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: okxwallet <122091289+okxwallet@users.noreply.github.com>
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