-
Notifications
You must be signed in to change notification settings - Fork 7
feat: add grouped exposure views #624
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
Open
Seranged
wants to merge
38
commits into
feature/lite-215-convert-vault-page-sections-to-accordion-style
Choose a base branch
from
feature/lite-216-add-open-interest-section-to-vault-page-sankey-style-flow
base: feature/lite-215-convert-vault-page-sections-to-accordion-style
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+2,305
−294
Open
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
2495bdd
feat: add vault open interest overview
Seranged b009aee
fix: polish open interest chart styling
Seranged 9810fb7
fix: refine open interest graph layout
Seranged 147be25
fix: move open interest into matrix view
Seranged d040f30
fix: collapse open interest matrix rows
Seranged 987a01e
fix: collapse open interest backing rows
Seranged 371a6b1
fix: center open interest show more
Seranged afed198
fix: add open interest token avatars
Seranged 958a177
fix: remove collateral availability datapoint
Seranged 7b8b4af
feat: group vault exposure surfaces
Seranged be8229e
fix: flatten single exposure groups
Seranged 7a48633
fix: trim compact exposure rows
Seranged c7f20e2
fix: remove exposure group headers
Seranged 8ab11c4
feat: add borrow open interest chart
Seranged 2551897
fix: remove open interest timestamp
Seranged a6a1c25
fix: stack collateral exposure metrics
Seranged 175c6fb
feat: add vault open interest chart
Seranged 7ff50f1
fix: remove matrix open interest timestamp
Seranged fb0f1f9
feat: merge exposure views
Seranged 7ddcf5c
fix: remove matrix open interest view
Seranged 5969070
fix: guard live exposure unavailable states
Seranged 943b855
fix: clarify exposure summaries
Seranged f7644db
fix: simplify lend collateral exposure
Seranged fa4d0e4
fix: clarify lend row backing asset
Seranged c4b609e
fix: align earn row exposure summary
Seranged b95d54c
fix: show live exposure in collateral modal
Seranged b1535fe
fix: preview collateral exposure modal on hover
Seranged 22e623c
fix: share collateral open interest loading
Seranged 80f2366
fix: remove earn vault combined exposure card
Seranged 5058e10
change UI
kanvgupta 429ea7e
fix: key exposure display rows by label
Seranged e9bc4bf
Merge pull request #657 from euler-xyz/pr624/review
Seranged 04e01cf
fix: harden exposure display states
Seranged 792e9e7
fix: show unavailable matrix exposure state
Seranged b5ea27f
fix: separate exposure market links
Seranged 8c4facf
fix: hide unresolved exposure market sources
Seranged 6b50ec0
fix: tighten exposure market label spacing
Seranged 243c4e4
fix: align exposure market links inline
Seranged File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Non-blocking maintainability note: this same
getStrategyMarketSourcehelper is now repeated across the Earn row, Earn stats, and Earn exposure components. The helper also drops the source entirely whengetProductKeyByVault()has no route key, even though the exposure sources modal can render a non-linked source label. If suppressing unlinked sources is intentional, fine; otherwise consider extracting a shared helper that always returns the label and conditionally addsto, with a small linked-vs-unlinked strategy test. Neat little branch, but it would like a single home.