-
Notifications
You must be signed in to change notification settings - Fork 45
feat: loan management form add/remove collateral #1715
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
0xPearce
wants to merge
65
commits into
main
Choose a base branch
from
feat/add-collateral-form
base: main
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.
+467
−226
Open
Changes from all commits
Commits
Show all changes
65 commits
Select commit
Hold shift + click to select a range
5120611
feat: layer 1 fill to collateral sub tabs
0xPearce c2f1d4f
feat: add collateral usd rate for input and token balance
0xPearce 49c3571
feat: input balance usd
0xPearce 1fb7656
refactor: loan info accordion with prev and current helper
0xPearce 23ce0e1
feat: previous collateral, debt, ltv and health to form
0xPearce 2b8fffb
refactor: handle prev and current values in the action info
0xPearce 6c2bc46
fix: token address already handled by validation suite
0xPearce cebd7b7
feat: added collateral greater than wallet balance error to validatio…
0xPearce 9dcfe73
feat: improve add collateral button label
0xPearce 1e876e2
fix: add collateral approve estimate gas query
0xPearce b99b20d
feat: remove min height from AppFormContentWrapper
0xPearce d098bf6
feat: fire icon to transaction action info
0xPearce ec65aa4
feat: layer 1 fill subtabs for lend markets
0xPearce 0486409
fix: review minor changes and optimizations
0xPearce 257b2d4
fix: sub tabs labels
0xPearce ad4f8af
fix: validation user collateral triggered when user collateral is 0
0xPearce 374a0e7
feat: loan accordion spacing
0xPearce 1fe14fa
feat: show usd rate when no input value
0xPearce 6adf3ed
refactor: form alert position
0xPearce c4949b0
feat: resolved custom alert form errors
0xPearce 9122063
feat: removed error notification
0xPearce 62afec5
refactor: expected collateral using bignumber
0xPearce c0cffef
feat: general error message for toast
0xPearce ead4f8b
Merge branch 'main' into feat/add-collateral-form
0xPearce 74f5cf4
refactor: get message error helper for loan and lend into llamalend
0xPearce 8fabbbf
refactor: use getErrorMessage from llamalend
0xPearce 1875f8e
Merge branch 'main' into feat/add-collateral-form
0xPearce 0710721
chore: format
0xPearce 737110d
refactor: pass user state to the loan info accordion
0xPearce 6714195
chore: docs and small optimization
0xPearce 4da5349
Merge branch 'main' into feat/add-collateral-form
0xPearce f4905cb
refactor: moved withTokenSymbol to llamalend utils
0xPearce ee3c124
fix: pass expectedBorrowed to loan to value calculation
0xPearce 2099feb
refactor: expected collateral calculation
0xPearce 40891a8
feat: added prev LTV, LTV, expected collateral and prev health to Rem…
0xPearce 65b645e
feat: custom position balance to LoanFormTokenInput
0xPearce 8661665
feat: custom position balance to LTI stories
0xPearce cd58063
refactor: position balance
0xPearce 0aefe06
fix: position balance array deps freezing loan form token input
0xPearce 7832bd6
refactor: collateral form validation suite with custom position for a…
0xPearce 96a7850
Merge branch 'main' into feat/add-collateral-form
0xPearce b365dd7
Merge branch 'feat/add-collateral-form' into feat/remove-collateral-form
0xPearce cecc493
fix: collateral sub tabs naming and color
0xPearce a23fcab
fix: token input freeze by enabling queries with form.isValid
0xPearce e8052eb
fix: enable new LTV query when user collateral value exist
0xPearce 2910e85
Merge branch 'main' into feat/add-collateral-form
0xPearce 0a4a733
refactor: hooks moved into useAddCollateralForm
0xPearce 6ae504e
Merge branch 'feat/add-collateral-form' into feat/remove-collateral-form
0xPearce 89fc65b
refactor: hooks moved into useRemoveCollateralForm
0xPearce 06ab6c4
Merge branch 'main' into feat/add-collateral-form
0xPearce b44d053
Merge branch 'feat/add-collateral-form' into feat/remove-collateral-form
0xPearce c88be3b
refactor: LoanInfoAccordion ternary and gaps
0xPearce c390786
feat: combine gas estimation for approval and add collateral if not a…
0xPearce fff1f4e
refactor: deduce userState type from query
0xPearce 1cddcf3
fix: check for empty string in Action Info
0xPearce 7b9be3a
refactor: tokenSymbol part of query's data
0xPearce 4ba0d56
refactor: combine query state
0xPearce 6ee365c
Merge branch 'main' into feat/add-collateral-form
0xPearce 40f91f8
Merge branch 'feat/add-collateral-form' into feat/remove-collateral-form
0xPearce 64c86d0
fix: tokenSymbol part of query's data
0xPearce 88f912a
refactor: hardcode LlamaIcon for position balance and improve code re…
0xPearce 2a4c061
Merge pull request #1748 from curvefi/feat/remove-collateral-form
0xPearce 7e6d82c
refactor: improve code readibility for collateral form hooks + docs
0xPearce c223982
Merge branch 'main' into feat/add-collateral-form
0xPearce 4a56ff7
feat: remove duplicate max collateral LTI message
0xPearce 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
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
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.
Uh oh!
There was an error while loading. Please reload this page.