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

build: remove idle and debt #68

Closed
wants to merge 3 commits into from
Closed

build: remove idle and debt #68

wants to merge 3 commits into from

Conversation

Schlagonia
Copy link
Collaborator

@Schlagonia Schlagonia commented Sep 27, 2023

Description

No longer track totalIdle and totalDebt seperatly.

Just track totalAssets and then use asset.balanceOf to get the loose funds when needed.

This reduces gas and allows for the allocation between debt and idle to be changed by the strategy without using one of the core TokenizedStrategy functions.

Fixes # (issue)

Checklist

  • I have run solidity linting
  • I have run the tests on my machine
  • I have followed commitlint guidelines
  • I have rebased my changes to the latest version of the main branch

@Schlagonia Schlagonia marked this pull request as draft September 27, 2023 03:25
@Schlagonia Schlagonia changed the base branch from master to v3.0.2 December 20, 2023 22:28
@Schlagonia
Copy link
Collaborator Author

moved here #73

@Schlagonia Schlagonia closed this Dec 21, 2023
@Schlagonia Schlagonia deleted the assets branch December 21, 2023 00:01
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.

1 participant