Feat/add okx wallet #1469
Merged
Feat/add okx wallet #1469
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Contributor
|
@okxwallet thank you for the PR and almost everything works well! |
Contributor
|
@okxwallet Hey just checking in here to see if you were able to resolve the issue raised by @Adamj1232? |
Adamj1232
approved these changes
Jan 27, 2023
Contributor
There was a problem hiding this comment.
Thank you again @okxwallet for the collaboration and PR to support OKX's new wallet!
If interested in doing a bit of marketing on the new support please feel free to reach out 😃
This was referenced Jan 27, 2023
Merged
Merged
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 <[email protected]> * 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 <[email protected]> * chore: update version Co-authored-by: Adam Carpenter <[email protected]> * docs: resolve typo Co-authored-by: Adam Carpenter <[email protected]> * fix: incorrectly checking providers * Add trust to docs site * Add docs and bump version * Fix deploy config --------- Co-authored-by: Adam Carpenter <[email protected]> * 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] <[email protected]> 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] <[email protected]> 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] <[email protected]> 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] <[email protected]> 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] <[email protected]> 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 <[email protected]> * Update versions for release * Address a couple of warnings from type-check * Bump sdk version in core --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: QiAbc <[email protected]> Co-authored-by: karim-yassine <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: okxwallet <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Checklist
package.jsonof the package you have made changes in is incremented following semantic versioning and using alpha release taggingyarn file-check,yarn type-check&yarn buildto confirm there are not any associated errorsIf 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 devto get the project running athttp://localhost:8080/Tests with demo app (injected)
Tests with demo app (SDK)