feat: add guide to Dashboard page#5258
Merged
therealemjy merged 1 commit intofeat/v4-phase-2from Feb 10, 2026
Merged
Conversation
🦋 Changeset detectedLatest commit: 94388e1 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
therealemjy
force-pushed
the
feat/dashboard-guide
branch
from
0030779 to
3686ed2
Compare
therealemjy
force-pushed
the
feat/dashboard-guide
branch
from
3686ed2 to
5a9f18e
Compare
therealemjy
force-pushed
the
feat/dashboard-summary
branch
from
0139ee3 to
34b49d8
Compare
therealemjy
force-pushed
the
feat/dashboard-guide
branch
from
5a9f18e to
4dd12f6
Compare
therealemjy
force-pushed
the
feat/dashboard-guide
branch
from
4dd12f6 to
ea9dfc0
Compare
therealemjy
force-pushed
the
feat/dashboard-summary
branch
from
34b49d8 to
1034958
Compare
| "guide": { | ||
| "completed": "已完成", | ||
| "step1": { | ||
| "description": "通过供应资产或将现有头寸迁移到 Venus 协议来赚取收益。", |
Contributor
There was a problem hiding this comment.
This Chinese translation looks weird. Might need to consult Zad for a more accurate translation.
| "guide": { | ||
| "completed": "已完成", | ||
| "step1": { | ||
| "description": "透過供應資產或將現有部位遷移到 Venus 協議來賺取收益。", |
| "title": "供应 / 迁移" | ||
| }, | ||
| "step2": { | ||
| "description": "借入资产增强您的 DeFi 策略，或一键使用流动性进行杠杆操作。", |
Contributor
There was a problem hiding this comment.
I believe there should be better wording in Chinese, since this is too obvious for auto-translated.
|
|
||
| return ( | ||
| <> | ||
| {/* Mobile UI */} |
Contributor
There was a problem hiding this comment.
Why not using conditional rendering instead? Having both rendered seems unnecessary and reduce performance especially the Carousel.
Member
Author
There was a problem hiding this comment.
I agree it makes sense here.
In general I use CSS, even if it means duplicating some DOM, just to stick to the Tailwind way of doing things. But in this case it's true that it means we display a heavy component for nothing. I'll update the code.
therealemjy
force-pushed
the
feat/dashboard-guide
branch
from
ae7cb03 to
133850e
Compare
zed-venus
reviewed
Feb 9, 2026
zed-venus
reviewed
Feb 9, 2026
zed-venus
reviewed
Feb 9, 2026
Member
Author
|
Thank you guys, I've updated the translations according to your suggestions. |
therealemjy
force-pushed
the
feat/dashboard-guide
branch
from
133850e to
edfd0fd
Compare
zed-venus
reviewed
Feb 9, 2026
therealemjy
force-pushed
the
feat/dashboard-guide
branch
from
edfd0fd to
d67191a
Compare
therealemjy
force-pushed
the
feat/dashboard-guide
branch
from
d67191a to
94388e1
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Jira ticket(s)
VPD-415
Changes
evm app