Add backwards-compatible redirect for /recommendation route #785
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Redirect /recommendation to /account with preserved query params - Ensures old links continue to work after integration into account screen
davidleomay
approved these changes
Dec 17, 2025
TaprootFreak
deleted the
feature/recommendation-redirect-compatibility
branch
TaprootFreak
added a commit
that referenced
this pull request
Dec 17, 2025
* Integrate recommendation functionality into account screen - Move recommendation management (invitations, pending requests) to account page - Show recommendations section only for KYC level 50+ users - Add modal for creating new invitations - Remove standalone /recommendation route and screen - Add bottom spacer to prevent background image overlap * [NO-TASK] Restructuring * [NO-TASK] UI improvements * Add backwards-compatible redirect for /recommendation route (#785) - Redirect /recommendation to /account with preserved query params - Ensures old links continue to work after integration into account screen --------- Co-authored-by: David May <david.leo.may@gmail.com>
TaprootFreak
added a commit
that referenced
this pull request
Dec 17, 2025
…ch , Integrate recommendation functionality into account screen (#786) * Fix default asset selection to prevent blockchain mismatch (#783) - Buy: Only auto-select first asset when activeBlockchain is known - Sell: Only auto-select first asset when activeBlockchain is known - Swap: Only auto-select source asset when walletBlockchain is known - Swap: Only auto-select target asset when blockchain is known Prevents 'Asset blockchain mismatch' and 'wallet type must match' errors when user's wallet blockchain differs from auto-selected asset's blockchain * Integrate recommendation functionality into account screen (#780) * Integrate recommendation functionality into account screen - Move recommendation management (invitations, pending requests) to account page - Show recommendations section only for KYC level 50+ users - Add modal for creating new invitations - Remove standalone /recommendation route and screen - Add bottom spacer to prevent background image overlap * [NO-TASK] Restructuring * [NO-TASK] UI improvements * Add backwards-compatible redirect for /recommendation route (#785) - Redirect /recommendation to /account with preserved query params - Ensures old links continue to work after integration into account screen --------- Co-authored-by: David May <david.leo.may@gmail.com> * Display user profile on account screen (#781) * Display user profile on account screen - Show email, name, address on /account page - Add profile section with KYC data (when available) - Add translations for DE, FR, IT * Use new organization object structure for profile display - Update to use profile.organization.name instead of profile.organizationName * Fix organization display to use organizationName - Change profile.organization?.name to profile.organizationName - Align with API response structure from PR #2637 * Add react-device-detect dependency * Update DFX packages to beta.214 and add react-icons * Add error handling for loadProfile() * Remove unused dependencies (react-device-detect, react-icons) * Extract address formatting to helper function * Add error logging for profile loading * Re-add react-icons (used in support, chat, kyc screens) --------- Co-authored-by: David May <david.leo.may@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adds a redirect from
/recommendationto/accountto ensure backwards compatibility after integrating the recommendation feature into the account screen.This addresses the concern raised in #780 comment about updating URLs in info mails - with this redirect, existing links will continue to work.
Changes
/recommendationnow redirects to/accountwith query parameters preserved/payment-link→/plredirectTest plan
/recommendation→ should redirect to/account/recommendation?code=test→ should redirect to/account?code=test