Skip to content

Fix Send not navigating correctly#3653

Merged
atn4z7 merged 2 commits intomainfrom
fix-send
Mar 17, 2026
Merged

Fix Send not navigating correctly#3653
atn4z7 merged 2 commits intomainfrom
fix-send

Conversation

@atn4z7
Copy link
Collaborator

@atn4z7 atn4z7 commented Mar 17, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 17, 2026 17:47
onghwan
onghwan previously approved these changes Mar 17, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Portfolio “Send” entry points to open the Send modal on the Recent Contacts screen instead of going directly to the Send form, aligning with the intended send flow.

Changes:

  • Portfolio home “Send” now navigates to /send/recentContacts
  • Token detail “Send” now navigates to /send/recentContacts (still passing vmName)

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
packages/core-mobile/app/new/features/portfolio/screens/PortfolioScreen.tsx Changes the Portfolio home Send navigation target to recent contacts.
packages/core-mobile/app/new/features/portfolio/assets/screens/TokenDetailScreen.tsx Changes the token detail Send navigation target to recent contacts while preserving vmName param passing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@atn4z7 atn4z7 dismissed stale reviews from onghwan and ruijialin-avalabs via 44ba3b5 March 17, 2026 18:05
@atn4z7 atn4z7 changed the title Fix Send not navigating to recent contacts Fix Send not navigating correctly Mar 17, 2026
@atn4z7 atn4z7 merged commit fe90d05 into main Mar 17, 2026
4 checks passed
@atn4z7 atn4z7 deleted the fix-send branch March 17, 2026 18:19
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.

4 participants