-
Notifications
You must be signed in to change notification settings - Fork 7
Add support for vault fee recipients #119
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
Merged
juangm
merged 20 commits into
main
from
pawel/aave-995-integrate-split-revenue-into-the-sdk
Oct 8, 2025
Merged
Changes from 5 commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
c5b4a44
Integrate split revenue into the sdk
desfero ff986e0
Merge branch 'main' into pawel/aave-995-integrate-split-revenue-into-…
desfero 5a1f404
Add changeset
desfero b17e19c
Update test name
desfero ae5c441
Fix lints
desfero fff6f0b
Fix lints
desfero 71573a7
Merge branch 'main' into pawel/aave-995-integrate-split-revenue-into-…
desfero feaf7c6
Update tests to match latest requirements with aave as one of default…
desfero b7d6df6
Merge branch 'main' into pawel/aave-995-integrate-split-revenue-into-…
juangm 41354e2
Merge branch 'main' into pawel/aave-995-integrate-split-revenue-into-…
desfero dca5a91
feat: implement actions and hooks
juangm 4c4b6f5
Merge branch 'main' into pawel/aave-995-integrate-split-revenue-into-…
juangm 1aeadc4
fix: refactor
juangm e130313
Update vault revenue splitter naming
desfero b920f14
test: add transfer ownership vault and set new recipients
juangm aac021d
fix: tests
juangm 5c18b6d
fix: remove console.logs
juangm 53743a1
fix: make internal set recipients in vaults
juangm c82c040
fix: remove not needed import
juangm b15b74c
fix: set actions as internal
juangm 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "@aave/graphql": patch | ||
| --- | ||
|
|
||
| **feat**: Add fee recipients support to the vault |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -680,6 +680,8 @@ enum OperationType { | |
| VAULT_FEE_WITHDRAWN | ||
| VAULT_WITHDRAW | ||
| LIQUIDATION | ||
| REVENUE_SPLITTER_OWNER_DEPLOYED | ||
| REVENUE_SPLITTER_RECIPIENT_SET | ||
| } | ||
|
|
||
| enum OrderDirection { | ||
|
|
@@ -941,6 +943,24 @@ type Query { | |
| savingsGhoWithdraw(request: SavingsGhoWithdrawRequest!): ExecutionPlan! | ||
| } | ||
|
|
||
| type RecipientPercent { | ||
| """Address of the recipient.""" | ||
| address: EvmAddress! | ||
|
|
||
| """ | ||
| Percentage of the fee that will be sent to the recipient, expressed in basis points. | ||
| """ | ||
| percent: PercentValue! | ||
| } | ||
|
|
||
| input RecipientPercentInput { | ||
| """Address of the recipient.""" | ||
| address: EvmAddress! | ||
|
|
||
| """Percentage of the fee that will be sent to the recipient.""" | ||
| percent: BigDecimal! | ||
| } | ||
|
|
||
| input RepayAmountInput @oneOf { | ||
| """The native amount""" | ||
| native: BigDecimal | ||
|
|
@@ -1708,9 +1728,6 @@ type Vault { | |
| """The vault address""" | ||
| address: EvmAddress! | ||
|
|
||
| """The owner of the vault""" | ||
| owner: EvmAddress! | ||
|
|
||
| """The name of the share""" | ||
| shareName: String! | ||
|
|
||
|
|
@@ -1737,11 +1754,15 @@ type Vault { | |
| """The user's shares info on the vault""" | ||
| userShares: UserVaultShares | ||
|
|
||
| """The owner of the vault""" | ||
| owner: EvmAddress! | ||
|
|
||
| """The total fees the owner can claim currently for the vault""" | ||
| feesBalance: TokenAmount! | ||
|
|
||
| """The total vault APR after their fee is taken off""" | ||
| vaultApr: PercentValue! | ||
| feeRecipients: [RecipientPercent!]! | ||
|
||
| } | ||
|
|
||
| input VaultDeployRequest { | ||
|
|
@@ -1775,6 +1796,12 @@ input VaultDeployRequest { | |
| The initial amount of underlying assets to deposit. This must be a non-zero, non-trivial amount, depending on the underlying asset's decimals. | ||
| """ | ||
| initialLockDeposit: BigDecimal! | ||
|
|
||
| """ | ||
| The recipients of the fee revenue, expressed as a percentage of the total fee revenue. | ||
| If not provided, the fee revenue will be sent to the owner of the vault. | ||
| """ | ||
| recipients: [RecipientPercentInput!] | ||
| } | ||
|
|
||
| input VaultDepositAmountInput { | ||
|
|
||
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
Large diffs are not rendered by default.
Oops, something went wrong.
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.