Skip to content

update block number calculation for query filter - #3443

Merged
portuu3 merged 1 commit into
developfrom
fix/gov-banner2
Jul 3, 2025
Merged

update block number calculation for query filter#3443
portuu3 merged 1 commit into
developfrom
fix/gov-banner2

Conversation

@portuu3

@portuu3 portuu3 commented Jul 3, 2025

Copy link
Copy Markdown
Collaborator

Issue tracking

NA

Context behind the change

Change the block number from which the governance logs are fetch to avoid timeouts.
Since the blocktime in polygon is 2s, voting delay is 1 day (86400s) and the duration of proposals is1 day (86400s), we won't need
more than 86400/2+86400/2= 86400 blocks and we rounded it 100k.

How has this been tested?

Run locally

Release plan

NA

Potential risks; What to monitor; Rollback plan

NA

@portuu3
portuu3 requested a review from flopez7 July 3, 2025 07:14
@portuu3 portuu3 self-assigned this Jul 3, 2025
@portuu3 portuu3 added the bug Something isn't working label Jul 3, 2025
@vercel

vercel Bot commented Jul 3, 2025

Copy link
Copy Markdown

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

Name Status Preview Comments Updated (UTC)
human-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 3, 2025 7:14am
4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
faucet-frontend ⬜️ Ignored (Inspect) Jul 3, 2025 7:14am
faucet-server ⬜️ Ignored (Inspect) Jul 3, 2025 7:14am
human-dashboard-frontend ⬜️ Skipped (Inspect) Jul 3, 2025 7:14am
staking-dashboard ⬜️ Skipped (Inspect) Jul 3, 2025 7:14am

@portuu3 portuu3 moved this to In Review in Core-tech - 2025 Jul 3, 2025
@portuu3
portuu3 merged commit e792064 into develop Jul 3, 2025
13 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in Core-tech - 2025 Jul 3, 2025
@portuu3
portuu3 deleted the fix/gov-banner2 branch July 3, 2025 07:18
@portuu3 portuu3 mentioned this pull request Jul 3, 2025
7 tasks
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

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants