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: bako sign message #390

Draft
wants to merge 107 commits into
base: main
Choose a base branch
from
Draft

feat: bako sign message #390

wants to merge 107 commits into from

Conversation

guimroque
Copy link
Contributor

No description provided.

guimroque added 30 commits March 19, 2024 13:13
@guimroque
fix
47ed191
@guimroque
feat(connector): bako safe
39f6324
@guimroque
fix(packages): remove not used packages and remove peerDependencies
3e33666
@guimroque
fix(consts): add url to bako safe api using domain .bako.global
ac600d3
@guimroque
fix(types): use typer imported of fuel
42771b8
@guimroque
feat(networks): implement method
352647c
@guimroque
fix(logo): use logo by bako.global domain
c602928
@guimroque
fix(disconnect): validate valid connection by verification on socket …
86e143f 
…started and api request
@guimroque
fix(packages): fixed version
0e16be5
@guimroque
fix(packages): fixed version
4d2d123
@guimroque
chore(): get infra updated changes
e4500b1
@guimroque
merge
f0a3fa7
@guimroque
fix: packages
38132f4
@guimroque
fix: remove axios and use fetch
70351cb
@guimroque
fix: remove axios to use fetch and use fuel storageAbstract
fed88a6
@guimroque
chore: changeset
3230f5e
@guimroque
fix: connection popup closing
fe25745
@guimroque
chore: pull main updates
fb71de8
@guimroque
chore: merge connector ui
3dc7aea
@guimroque
chore: merge main
2c06894
@guimroque
Merge branch 'main' of github.com:FuelLabs/fuel-connectors into gr/fe…
1e5cbb7 
…at/bakosafe
@guimroque
fix: return an error with window of transaction has closed
9d830e8
@guimroque
fix: move local events emit set to setup
3470c03
@guimroque
fix: verify if event has origin on bako server
6170de5
@guimroque
chore: pull main
a9fc075
@guimroque
Merge branch 'main' of github.com:FuelLabs/fuel-connectors into gr/fe…
d5369cc 
…at/bakosafe
@guimroque
fix: transaction and call contract request
37ea5af
@guimroque
wip: socket
e3554dc
@guimroque
wip: safari
4b17500
@guimroque
wip: ping method and message control by id
86d2050
Voxelot
Voxelot reviewed Oct 14, 2024
View reviewed changes
packages/bako-safe/src/BakoSafeConnector.ts Outdated Show resolved Hide resolved
packages/bako-safe/src/BakoSafeConnector.ts Outdated Show resolved Hide resolved
packages/bako-safe/src/BakoSafeConnector.ts Outdated Show resolved Hide resolved
.github/workflows/pr-release.yaml Outdated Show resolved Hide resolved
@guimroque guimroque requested a review from Voxelot October 14, 2024 01:24
@guimroque guimroque mentioned this pull request Oct 14, 2024
Merged
@vercel Vercel
Copy link

vercel bot commented Oct 14, 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 Oct 14, 2024 3:53am

@LuizAsFight
Copy link
Contributor

when I click to sign, I get a dns error

image

LuizAsFight
LuizAsFight requested changes Oct 14, 2024
View reviewed changes
Copy link
Contributor

@LuizAsFight LuizAsFight left a comment

Choose a reason for hiding this comment

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

dns error when asking to sign.

PS: now you can test signing directly in connectors react app example. added in #392

@guimroque
Copy link
Contributor Author

guimroque commented Oct 14, 2024 •
edited
Loading

when I click to sign, I get a dns error

image

yes, I ended up forgetting the / in the redirect url for the bako ui /dapp/sign/:message

fixed here: 321ab5da8a4428b4ba251d11432446245ef65cf7

@guimroque guimroque marked this pull request as draft October 14, 2024 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maschad maschad maschad left review comments

@LuizAsFight LuizAsFight LuizAsFight requested changes

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

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

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

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

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

@Voxelot Voxelot Awaiting requested review from Voxelot

Requested changes must be addressed to merge this pull request.

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

Successfully merging this pull request may close these issues.
4 participants
@guimroque @LuizAsFight @Voxelot @maschad