Skip to content

Conversation

@thekidnamedkd
Copy link
Contributor

@thekidnamedkd thekidnamedkd commented Sep 25, 2025

Description

Implement useTokenCurrentDelegate hook, refactor tokenDelegationForm

Task: APP-4588

Type of Change

  • New feature (non-breaking change which adds functionality)

Developer Checklist:

  • Manually smoke tested the functionality locally
  • Confirmed there are no new warnings or errors in the browser console
  • (For User Stories only) Double-checked that all Acceptance Criteria are satisfied
  • Confirmed there are no new warnings on automated tests
  • Merged and published any dependent changes in downstream modules
  • Selected the correct base branch
  • Commented the code in hard-to-understand areas
  • Followed the code style guidelines of this project
  • Reviewed that the Files Changed in Github’s UI reflect my intended changes
  • Confirmed the pipeline checks are not failing

Review Checklist:

  • Tested locally that all Acceptance Criteria or Expected Outcomes are satisfied
  • Confirmed that changes follow the code style guidelines of this project

@thekidnamedkd thekidnamedkd changed the title WIP fix(APP-4588): Implement useTokenCurrentDelegate hook WIP fix(APP-4588): Implement useTokenCurrentDelegate hook to fix UX on TokenDelegationForm Sep 25, 2025
@thekidnamedkd thekidnamedkd requested review from cgero-eth and milosh86 and removed request for cgero-eth September 25, 2025 12:59
@thekidnamedkd thekidnamedkd changed the title WIP fix(APP-4588): Implement useTokenCurrentDelegate hook to fix UX on TokenDelegationForm fix(APP-4588): Implement useTokenCurrentDelegate hook to fix UX on TokenDelegationForm Sep 25, 2025
@milosh86 milosh86 self-assigned this Sep 25, 2025
@github-actions
Copy link

github-actions bot commented Sep 25, 2025

🚀 Preview Deployment: View Here

Copy link
Contributor

@milosh86 milosh86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a couple of small comments and I'm still left to test it in the preview, but looks great overall 👍

@milosh86 milosh86 removed their assignment Sep 29, 2025
@cgero-eth cgero-eth merged commit 4b4d02c into main Sep 29, 2025
10 checks passed
@cgero-eth cgero-eth deleted the fix/APP-4588 branch September 29, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants