Add bearby wallet integration #264
Conversation
mazmassa
changed the title
mazmassa
changed the title
| @@ -33,13 +31,13 @@ export const U256_MAX = BigInt( | |||
| export const forwardBurnFees = { | |||
| fee: 0n, | |||
| coins: 100000n, | |||
| maxGas: 1000000n, | |||
| maxGas: 4294967295n, | |||
There was a problem hiding this comment.
There was a problem hiding this comment.
i don't think you should change this.
There was a problem hiding this comment.
That's why I place an alert here. ;)
However, with 1000000n it just didn't work (was not working at least). So, in the received error that was the amount asked to fill.
There was a problem hiding this comment.
I think it used to work because station set the maximum gas level itself.
There was a problem hiding this comment.
Today I'm not 100% sure about that but it could explain why the gas amount was ok with station but not bearby
| }; | ||
|
|
||
| export const increaseAllowanceFee = { | ||
| fee: 0n, | ||
| coins: 1000n, | ||
| maxGas: 1000000n, | ||
| maxGas: 4294967295n, |
There was a problem hiding this comment.
mazmassa
force-pushed
the
add-bearby-integration
branch
2 times, most recently
from
12372ce to
9cc0bc5
Compare
|
Default value or placeholder is missing! |
Thykof
changed the title
Thykof
force-pushed
the
add-bearby-integration
branch
4 times, most recently
from
30df08a to
21b499c
Compare
peterjah
force-pushed
the
add-bearby-integration
branch
from
21b499c to
784d886
Compare
| .addString(CONTRACT_ADDRESS) | ||
| .addU256(amount) | ||
| .serialize(), | ||
| parameter: new Args().addString(CONTRACT_ADDRESS).addU256(amount), |
There was a problem hiding this comment.
maybe this is why the operations don't work with bearby
Thykof
force-pushed
the
add-bearby-integration
branch
from
784d886 to
cc10bb6
Compare
install missing dependencies fix typo NetworkStoreState cypress add support file rename supportedTokens preselect a wallet provider
Thykof
force-pushed
the
add-bearby-integration
branch
from
cc10bb6 to
523a0b5
Compare
No description provided.