Skip to content

feat: new flow done#63

Merged
fiqriBTI merged 3 commits intomainfrom
feat-new-app-flow
Nov 30, 2025
Merged

feat: new flow done#63
fiqriBTI merged 3 commits intomainfrom
feat-new-app-flow

Conversation

@fiqriBTI
Copy link
Collaborator

@fiqriBTI fiqriBTI commented Nov 23, 2025

  1. Remove "Wallet Not Connected" layout, this improves UX by showing user what web contains without having to connect their wallet
  2. Make all action button on every page shows "Connect Wallet" with action to open wallet list dialog if user hasn't connected to their wallet
  3. Implement notification "Please connect your wallet to see your balance/stats/position" on components like balance item, position pool card, and stats pool card
  4. Improved every services that needs to has "owner address" by not running it if user hasn't connected to their wallet instead showing "Please connect your wallet to see your balance/stats/position" notification
  5. Improved loading and refetching state to make smooth transition between not connected wallet and connected wallet in every page
  6. Change repository structure from using path params into query params in some pages like liquidity-pools/detail, liquidity-pools/deposit, and so on. This will make repo looks more readable and cleaner because of removing nested folder of page

@fiqriBTI fiqriBTI requested a review from philipdev1 November 23, 2025 18:14
@vercel
Copy link
Contributor

vercel bot commented Nov 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
bbachain-token Ready Ready Preview Comment Nov 24, 2025 6:07am

@fiqriBTI fiqriBTI merged commit 22514eb into main Nov 30, 2025
3 checks passed
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.

1 participant