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

1. minor update about borrow-erc20-with-eth-collateral 2. add borrowing based reinvestment pattern #4

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

michael-spengler
Copy link

@michael-spengler michael-spengler commented Jan 3, 2021

The first commit is just a minor simplification proposal. Your examples saved me a lot of research time --> Thank You.

The second (or subsequent commits) shall propose adding a borrowing based reinvestment pattern example. With this you can easily use / increase the leverage effect when you are e.g. betting on rising ether prices.

@michael-spengler michael-spengler changed the title Update borrow-erc20-with-eth-collateral.js 1. minor update about borrow-erc20-with-eth-collateral 2. add borrowing based reinvestment pattern Jan 4, 2021
@michael-spengler
Copy link
Author

michael-spengler commented Jan 8, 2021

@byrmarkn Thank you for approving the PR. Is there anything else necessary before this PR can be merged?

I'm considering to add a smart contract based implementation of this borrowing based reinvestment scenario to the solidity-examples folder. Would this be valuable from your side? Do you have any hints for me on how to do this in a good way?

I do not know exactly how long it would take me as I'm relatively new to solidity.

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