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.

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

Investigate stCELO "withdrawal" bug #200

Closed
3 tasks
arthurgousset opened this issue Jan 16, 2024 · 6 comments · Fixed by #235
Closed
3 tasks

Investigate stCELO "withdrawal" bug #200

arthurgousset opened this issue Jan 16, 2024 · 6 comments · Fixed by #235
Assignees

Comments

@arthurgousset
Copy link
Contributor

arthurgousset commented Jan 16, 2024

Copy/pasting bug report from Daniel on Slack

I am getting an error when I try to unstake on stCelo. I get a:

eth_signTransaction: Gas must be provided to sign a transaction

I do have CELO to pay for gas.
this is a screenshot from the console:

image

we reviewed this error with @therealharpaljadeja and concluded it might be a FrontEnd Issue. @therealharpaljadeja also noticed that the contracts are not verified on Celoscan

Tasks

@arthurgousset
Copy link
Contributor Author

arthurgousset commented Jan 16, 2024

This might be caused by a known bug in the stcelo bot. The bug leads to failed activate, claim, and vote operations and occurs when there is a race condition in the bot. It looks like this race condition is becoming more frequent with more traffic.

More context and fix by @celo-org/primitives in this stcelo bot PR:

@arthurgousset arthurgousset changed the title Investigate stCELO UI bug Investigate stCELO "withdrawal" bug Jan 16, 2024
@pahor167
Copy link
Contributor

This might be caused by a known bug in the stcelo bot. The bug leads to failed activate, claim, and vote operations and occurs when there is a race condition in the bot. It looks like this race condition is becoming more frequent with more traffic.

More context and fix by @celo-org/primitives in this stcelo bot PR:

This seems like problem with wallet or UI, since st-celo bot bug cause only the api call to revert but this looks like revert of original transaction in user's wallet.

@arthurgousset
Copy link
Contributor Author

Thanks for chiming in @pahor167 !

@arthurgousset
Copy link
Contributor Author

Copy/pasting follow-up questions from Slack for context:

I am using a Ledger Nano S and Celo Terminal. I connect it using Wallet Connect.

I am using Chrome

By the way something else odd is happening, one I connect with Wallet Connect and disconnect I am unable to do it again on the same browser account. When I try to connect again and click on Wallet Connect it doesn't show the QR code modal.

update: The contract was verified on CeloScan,
@therealharpaljadeja and I tried to use the withdraw function and got the same error. We now think this is a Celo Terminal issue.

@arthurgousset
Copy link
Contributor Author

@therealharpaljadeja: If you think this problem originates in Celo Terminal, have you reached out to @zviadm in zviadm/celoterminal yet?

@arthurgousset
Copy link
Contributor Author

Feedback from zviad:

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 a pull request may close this issue.

3 participants