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

Confirmations required always shows as 1 #2375

Open
SanLeo461 opened this issue Jan 6, 2025 · 5 comments
Open

Confirmations required always shows as 1 #2375

SanLeo461 opened this issue Jan 6, 2025 · 5 comments
Labels
bug Something isn't working

Comments

@SanLeo461
Copy link

SanLeo461 commented Jan 6, 2025

Just gonna paste in what I posted in the safe-wallet-monorepo issue:

Bug description

Transactions once signed & proposed by one user show as needing only 1 confirmation total and ready to be executed, while the true threshold is 4.
This is with regards to the safe at: https://nova.arbiscan.io/address/0x2C0BDb957F5a6976E4Ceb5A34470bFB13D30a278

If this is some bizarre indexing issue and you know where to override it for a temporary fix, that's good enough for me, however I couldn't find where the threshold was stored in any of the DBs.

Environment

  • Production
  • Browser: Chrome, on Windows & Linux
  • Wallet: MetaMask
  • Chain: Arbitrum Nova (42170)

Steps to reproduce

  1. Go to [Link redacted]

Expected result

Threshold shows as 4

Obtained result

Threshold shows as one and appears ready for execution

Screenshots

image

@SanLeo461 SanLeo461 added the bug Something isn't working label Jan 6, 2025
@on-waterfall
Copy link

I have the same bug.
I discovered that SafeStatus is not created in the database.
If I manually add it to the database, it works.

@SanLeo461
Copy link
Author

I have the same bug. I discovered that SafeStatus is not created in the database. If I manually add it to the database, it works.

Worked for me, thanks!
Took ages to figure out the syntax and get the data inserted though haha.

I'm gonna leave this open in case the team wants to investigate further on the issue or ask any questions of me, otherwise maintainers feel free to close.
Cheers

@on-waterfall
Copy link

But this is not a solution to add manually, it should be added automatically

@SanLeo461
Copy link
Author

But this is not a solution to add manually, it should be added automatically

Yeah it should be definitely, in my case though I only needed a one time fix. I'll leave this up and hopefully the safe team can fix it properly.

@on-waterfall
Copy link

I add processing to WORKER_QUEUES env and it works

WORKER_QUEUES=default,indexing,processing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants
@on-waterfall @SanLeo461 and others