Skip to content

Comments

feat/account-upgrade-flow#400

Open
Nodirbek75 wants to merge 102 commits intomainfrom
feat/business-account
Open

feat/account-upgrade-flow#400
Nodirbek75 wants to merge 102 commits intomainfrom
feat/business-account

Conversation

@Nodirbek75
Copy link
Contributor

@Nodirbek75 Nodirbek75 commented May 8, 2025

UI/UX for Account upgrade flow

@Nodirbek75 Nodirbek75 self-assigned this May 8, 2025
@Nodirbek75 Nodirbek75 marked this pull request as draft May 8, 2025 15:02
@islandbitcoin islandbitcoin linked an issue May 12, 2025 that may be closed by this pull request
@islandbitcoin
Copy link
Contributor

Feedback:

  • Cannot leave screen and then return to complete form with out it freezing.
  • Google Maps API not working on local build
  • Pending card should not have an X to close it out

@islandbitcoin islandbitcoin force-pushed the feat/business-account branch from 393e6d8 to 9fe16fe Compare May 17, 2025 20:14
@Nodirbek75
Copy link
Contributor Author

Waiting for the backend to implement the mutation for upgrading user level.

@islandbitcoin
Copy link
Contributor

Call mutation to add email address to account when user enters email in Personal Upgrade flow

@islandbitcoin
Copy link
Contributor

islandbitcoin commented Oct 16, 2025

Move Google API key to Flash server, and call a GraphqQL query to give business address enrichment

@islandbitcoin
Copy link
Contributor

subtask: Implement query on backend to facilitate data enrichment

@islandbitcoin islandbitcoin changed the title Account upgrade flow feat/business-account Dec 29, 2025
@islandbitcoin islandbitcoin changed the title feat/business-account feat/account-upgrade-flow Jan 27, 2026
@islandbitcoin
Copy link
Contributor

is this ready for review?

@Nodirbek75
Copy link
Contributor Author

is this ready for review?

No, waiting backend to fix supportedBanks query

@Nodirbek75
Copy link
Contributor Author

Faced issues:

  1. When a user requests an account upgrade to level PRO (TWO), it should be auto-approved. Right now it’s showing as Pending. We may need to update the logic so that requests with requestedLevel === THREE are set to Pending, and all others are set to Approved.
  2. When a user upgrades to level PRO and it’s auto-approved, then submits a new request to level MERCHANT, the status should switch back to Pending. However, it’s still returning Approved, and the requestedLevel is showing as TWO. I think the accountUpgradeRequest query is fetching the first request instead of the latest.
  3. When a user upgrades to PRO, deletes the app, logs back in, and then requests an upgrade to MERCHANT without uploading a new document, the existing document is not available in the admin panel for the new request.

@Nodirbek75
Copy link
Contributor Author

These changes fix the above issues: lnflash/flash#278
Waiting accountUpgradeRequest query update to do next end-to-end tests

@Nodirbek75
Copy link
Contributor Author

  • update button title on the account settings screen to align with quick card
  • alert message on the validation screen to let user know personal upgrade
  • make email optional for pro and merchant
  • edit button on the account screen and disable quick card button

@Nodirbek75 Nodirbek75 marked this pull request as ready for review February 23, 2026 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

2 participants