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.

Sign up for GitHub

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

Jump to bottom

feat: add Wallet Connect connector #61

Merged
merged 44 commits into from
Apr 18, 2024
Merged

feat: add Wallet Connect connector #61

merged 44 commits into from
Apr 18, 2024

Conversation

luizstacio
Copy link
Member

closes: #54, #53, #35 and #60

This PR was originally opened by @pedropereiradev here #43 the work here is just merging the main branch and removing MetaMask

pedropereiradev and others added 30 commits April 6, 2024 16:15
@pedropereiradev
feat: ethereum wallet connector implementation started
18f4edc
@pedropereiradev
feat: connector renamed and all methods implemented
0bf8d59
@pedropereiradev
fix: platform changed to browser
fab8817
@pedropereiradev
feat: walletconnect connector added to default connectors and evm con…
ae3c5df 
…nector temporarily removed
@pedropereiradev
build: pnpm-lock updated
78260be
@pedropereiradev
fix: connect method fixed
911b465
@pedropereiradev
fix: window.ethereum removed as provider
f98375f
@pedropereiradev
fix: disconnect method fixed to reload page as walletconnect needs
c30be18
@pedropereiradev
docs: changeset created
a699b8e
@pedropereiradev
test: same tests from evm connector implemented
6def350
@pedropereiradev
chore: remove unused files
f72d82c
@pedropereiradev
fix: evm connector added to default connectors again
3d11378
@pedropereiradev
fix: window.ethereum removed due conflicts with a dependency
862a85e
@pedropereiradev
refactor: walletconnect connector refactored
c2227ac
@pedropereiradev
fix: wallet disconnect and account address inconsistencies fixed
d494b3b
@pedropereiradev
fix: refactor project and fix bugs
517e200
@pedropereiradev
test: ignore tests
85766d8
@pedropereiradev
build: build process changed
fc8dc05
@pedropereiradev
fix: removed unrelated code
ead51c1
@pedropereiradev
fix: restore tsup values
a2d7bf5
@pedropereiradev
test: tests for not implemented methods created
4572fc7
@pedropereiradev
fix: size of connector icon fixed
ab8e26f
@pedropereiradev
test: silent console log in tests
def8168
@pedropereiradev
fix: walletconnect connector icon changed
1c07b47
@pedropereiradev
test: fix warn logs
bc4d09b
@pedropereiradev
fix: predicate hashTransaction fixed
5d5ced1
@pedropereiradev
fix: .env ignored
6d98124
@pedropereiradev
fix: old connector events removed
a8cd1b4
@pedropereiradev @luizstacio
Update examples/react-app/src/main.tsx
df9a676 
Co-authored-by: Luiz Estácio | stacio.eth <luizstacio@gmail.com>
@pedropereiradev
Merge branch 'feat/ethereum-wallets-connector' of github.com:pedroper…
4968e6f 
…eiradev/fuel-connectors into feat/ethereum-wallets-connector
@vercel Vercel
Copy link

vercel bot commented Apr 18, 2024 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
fuel-connectors ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 18, 2024 7:17pm

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 18, 2024 •
edited
Loading

Coverage Report for Fuel Wallet (./packages/fuel-wallet)

Status Category Percentage Covered / Total
🔵 Lines 72.29% 261 / 361
🔵 Statements 72.29% 261 / 361
🔵 Functions 53.33% 16 / 30
🔵 Branches 90.47% 19 / 21
File CoverageNo changed files found.
Generated in workflow #157

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 18, 2024 •
edited
Loading

Coverage Report for Fuel Development Wallet (./packages/fuel-development-wallet)

Status Category Percentage Covered / Total
🔵 Lines 100% 19 / 19
🔵 Statements 100% 19 / 19
🔵 Functions 100% 1 / 1
🔵 Branches 100% 1 / 1
File CoverageNo changed files found.
Generated in workflow #157

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 18, 2024 •
edited
Loading

Coverage Report for Fuelet Wallet (./packages/fuelet-wallet)

Status Category Percentage Covered / Total
🔵 Lines 100% 22 / 22
🔵 Statements 100% 22 / 22
🔵 Functions 100% 1 / 1
🔵 Branches 100% 1 / 1
File CoverageNo changed files found.
Generated in workflow #157

@luizstacio luizstacio mentioned this pull request Apr 18, 2024
Closed
@luizstacio luizstacio merged commit f56bd40 into main Apr 18, 2024
9 checks passed
@luizstacio luizstacio deleted the pr/43 branch April 18, 2024 19:26
@fuel-service-user fuel-service-user mentioned this pull request Apr 18, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@helciofranco helciofranco helciofranco approved these changes

@pedronauck pedronauck Awaiting requested review from pedronauck pedronauck is a code owner

@LuizAsFight LuizAsFight Awaiting requested review from LuizAsFight LuizAsFight is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Metamask not properly disconnecting
3 participants
@luizstacio @helciofranco @pedropereiradev