Skip to content

fix: finalize withdraw timestamp #11

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
Merged
aminsammara merged 2 commits into from
Feb 9, 2026
Merged

fix: finalize withdraw timestamp #11

aminsammara merged 2 commits into from
Feb 9, 2026

Conversation

@robertbrada
Copy link
Collaborator

@robertbrada robertbrada commented Feb 9, 2026 •
edited
Loading

fix https://linear.app/aztec-labs/issue/GK-553/fix-finalize-withdraw-timestamp

Summary

  • Fixed withdrawal timestamp handling across delegation and direct stake components
  • Added governance withdrawal hook
  • Enhanced sequencer status handling with improved timestamp logic
  • Updated date formatters for better timestamp display

Changes

  • Updated withdrawal actions in ATP details and wallet components
  • Added useGovernanceWithdrawal hook with timestamp handling
  • Improved useSequencerStatus hook with enhanced timestamp logic
  • Fixed date formatter utilities

@robertbrada robertbrada requested review from a team and aminsammara February 9, 2026 14:57
aminsammara
aminsammara previously approved these changes Feb 9, 2026
View reviewed changes
@robertbrada @claude
fix: require both exit and withdrawal for actualUnlockTime
d84f93e 
Changed fallback behavior to return undefined instead of exit.exitableAt
when withdrawal data is unavailable. Both values must be present to
accurately calculate the unlock time, preventing display of potentially
misleading incomplete data during loading states.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@aminsammara aminsammara merged commit 9f3d27b into main Feb 9, 2026
5 checks passed
@robertbrada robertbrada deleted the fix/withdraw-timestamp branch February 10, 2026 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@saleel saleel saleel approved these changes

@aminsammara aminsammara aminsammara left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@robertbrada @saleel @aminsammara