onflow / onflow/FlowActions

[IMPROVEMENT] Put Autobalancers into a separate contract

Open
#147 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Cadence
Stars
9
Forks
3
Avg merge
1h 45m
Merged PRs (30d)
2

Description

Issue to be solved

every time the autobalancers contract is updated, there is a chance to introduce non-upgradeable changes that affects FlowALP, doesn't use Autobalancers

autobalancers are used only in strategies so far and it's easier to version strategies, than the whole FlowALP contract

Suggest A Solution

put Autobalancers into a separate contract and remove it from DeFiActions.cdc

What are you currently working on that this is blocking?

No response

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading DeFiActions.cdc and tracing how Autobalancers are used by strategies and avoided by FlowALP. Move Autobalancers into a separate contract and remove them from DeFiActions.cdc. Done means strategy usage remains available while FlowALP no longer depends on Autobalancers.

Written by the indexing model from the issue text.

Assessment

Domain
blockchain
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.