Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trade Count #1561

Merged
merged 14 commits into from
Nov 5, 2024
Merged

Trade Count #1561

merged 14 commits into from
Nov 5, 2024

Conversation

GrandSchtroumpf
Copy link
Collaborator

fix #1336

  • Add Trade count on strategy card
  • Remove everything related to roi
image image

What to test :

  • Go to explorer page with pair ETH/USDC
  • Strategies should have a trade count
  • Click Filter/Sort
  • Default sort should be trade, which sort by descending order

Copy link

cloudflare-workers-and-pages bot commented Oct 30, 2024

Deploying carbon-app-blast with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7e2655a
Status: ✅  Deploy successful!
Preview URL: https://4e34be5d.carbon-app-blast.pages.dev
Branch Preview URL: https://issue--1336.carbon-app-blast.pages.dev

View logs

Copy link

cloudflare-workers-and-pages bot commented Oct 30, 2024

Deploying carbon-app-celo with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7e2655a
Status: ✅  Deploy successful!
Preview URL: https://530402e5.carbon-app-celo.pages.dev
Branch Preview URL: https://issue--1336.carbon-app-celo.pages.dev

View logs

Copy link

cloudflare-workers-and-pages bot commented Oct 30, 2024

Deploying carbon-app with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7e2655a
Status: ✅  Deploy successful!
Preview URL: https://256ff570.carbon-app-csq.pages.dev
Branch Preview URL: https://issue--1336.carbon-app-csq.pages.dev

View logs

Copy link

cloudflare-workers-and-pages bot commented Oct 30, 2024

Deploying carbon-app-sei with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7e2655a
Status: ✅  Deploy successful!
Preview URL: https://13ee1a0a.carbon-app-sei.pages.dev
Branch Preview URL: https://issue--1336.carbon-app-sei.pages.dev

View logs

Copy link
Collaborator

@tiagofilipenunes tiagofilipenunes left a comment

Choose a reason for hiding this comment

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

Looks good to me - just a minor comment about migrating the localstorage instead of keeping the roi sort enums

Also it might be best to mock the trade count route on e2e to prevent issues with screenshots

src/components/strategies/overview/StrategyFilterSort.tsx Outdated Show resolved Hide resolved
src/components/strategies/overview/StrategyFilterSort.tsx Outdated Show resolved Hide resolved
src/libs/queries/sdk/strategy.ts Show resolved Hide resolved
src/libs/queries/sdk/strategy.ts Show resolved Hide resolved
src/libs/queries/sdk/strategy.ts Show resolved Hide resolved
@ashachaf
Copy link
Collaborator

ashachaf commented Nov 3, 2024

  • update "trades" title
    current: "Trades"
    updated: "Trade Count" and remove the i icon and tool tip

  • remove "total budget" tool tip and i icon

  • update "buy {token}" tool tip
    current: This is the available amount of {quoteToken} tokens that you are willing to use in order to buy {baseToken}.
    updated: {quoteToken} tokens available to buy {baseToken}.

  • update "sell {token}" tool tip
    current: This is the available amount of {baseToken} tokens that you are willing to sell.
    updated: {baseToken} tokens available for sale.

@GrandSchtroumpf GrandSchtroumpf merged commit a286667 into main Nov 5, 2024
4 checks passed
@GrandSchtroumpf GrandSchtroumpf deleted the issue-#1336 branch November 5, 2024 08:47
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.

Strategy Overview - updating the data and design of the strategy overview card
3 participants