Skip to content

feat(APP-604): Add new SelectAddress component for asset selection#1143

Merged
milosh86 merged 23 commits into
mainfrom
app-604-new-selectaddress-component
May 26, 2026
Merged

feat(APP-604): Add new SelectAddress component for asset selection#1143
milosh86 merged 23 commits into
mainfrom
app-604-new-selectaddress-component

Conversation

@milosh86
Copy link
Copy Markdown
Contributor

Description

Introduces a new AssetAddressSelect component replacing the existing AssetSelectionDialog flow in AssetInput. Users can now pick a token from the DAO's asset list, search by name/symbol/address, or add a custom token by entering its address (resolved on the fly via RPC).

Linear: APP-604

Highlights

  • New AssetAddressSelect component with paginated list, search, and "add by address" flow
  • Subcomponents: AssetAddressSelectItem, AssetAddressSelectAddAddressView, AssetAddressSelectAddButton, AssetAddressSelectBackButton, AssetAddressSelectEmptyState
  • Search by name, symbol, or address; auto-resolves addresses immediately when the query is an address
  • AssetInput updated to use the new selector and to flip the max-amount layout per design
  • Token symbol/logo fallbacks tightened in AssetInputToken
  • Bumps @aragon/gov-ui-kit to use the new DataListItem select variant

Type of Change

  • Major: Breaking change (change that would cause existing functionality to not work as expected)
  • Minor: Feature (non-breaking change which adds new functionality)
  • Patch: Enhancement (non-breaking change to an existing feature)
  • Patch: Bug fix (non-breaking change which fixes an issue)

Developer Checklist:

  • Manually smoke tested the functionality in a preview or 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:

  • (For User Stories only) Tested in a preview or locally that all Acceptance Criteria are satisfied
  • Confirmed that changes follow the code style guidelines of this project

milosh86 added 9 commits May 18, 2026 17:05
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 20, 2026

APP-604

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 20, 2026

🚀 Preview Deployment: View Here

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 20, 2026

E2E results (preview)

Smoke

Base URL https://app-next-hpx0xa2cv-aragon-app.vercel.app
Suite smoke
Playwright result ✅ passed
Summary 75 passed, 5 skipped
HTML report View report
GitHub job ✅ completed

View run #4219

milosh86 added 2 commits May 20, 2026 16:33
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
@milosh86 milosh86 marked this pull request as ready for review May 20, 2026 18:15
@milosh86 milosh86 requested a review from a team as a code owner May 20, 2026 18:15
Copy link
Copy Markdown
Contributor

@thekidnamedkd thekidnamedkd left a comment

Choose a reason for hiding this comment

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

Looking good. Couple questions. 🧑‍🚀

Comment thread src/modules/finance/components/assetAddressSelect/assetAddressSelectAddButton.tsx Outdated
@@ -0,0 +1,35 @@
import { IconType } from '@aragon/gov-ui-kit';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't remember this pattern. It's sort of like hook/util as component? Naming and location seems off maybe.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

right, I was on the fence when I saw it and I just let it go, but you are right, no need to have a different pattern for the same thing we already have in other places.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

.... I just inlined empty state as it's passed as a prop


const pluginSetupRef = {
pluginSetupRepo,
pluginSetupRepo: pluginSetupRepo as Hex,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need to cast?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this was strange, I don't know why I've got this TS error, I didn't change anything related .... I'll double check, maybe just some TS cache issue

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

could you please check if you get this TS error in main? I do, and I don't know how we got around it before...

milosh86 added 2 commits May 21, 2026 17:23
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
@milosh86
Copy link
Copy Markdown
Contributor Author

milosh86 commented May 22, 2026

@thekidnamedkd there is also an additional govkit PR related to this one, so we need a release before merging

Signed-off-by: Milos Dzepina <milos@aragon.org>
Copy link
Copy Markdown
Contributor

@thekidnamedkd thekidnamedkd left a comment

Choose a reason for hiding this comment

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

Did a closer pass — overall this is shaping up nicely. A few things worth a look, the biggest being the decimals fallback on the Unknown asset row, which I think will throw downstream in the transfer flow.

Comment thread src/modules/finance/components/assetAddressSelect/assetAddressSelect.tsx Outdated
Comment thread src/modules/finance/components/assetAddressSelect/assetAddressSelectItem.tsx Outdated
milosh86 added 3 commits May 25, 2026 14:36
Signed-off-by: Milos Dzepina <milos@aragon.org>
Signed-off-by: Milos Dzepina <milos@aragon.org>
Copy link
Copy Markdown
Contributor

@thekidnamedkd thekidnamedkd left a comment

Choose a reason for hiding this comment

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

LGTM 🔥

@milosh86 milosh86 merged commit b4f77d4 into main May 26, 2026
17 checks passed
@milosh86 milosh86 deleted the app-604-new-selectaddress-component branch May 26, 2026 08:39
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 this pull request may close these issues.

2 participants