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.

Sign up for GitHub

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

Jump to bottom

[Volume Splitting] refactor: remove even splitting policy #6500

Closed
Tracked by #6057
p0mvn opened this issue Sep 23, 2023 · 0 comments · Fixed by #6512
Closed
Tracked by #6057

[Volume Splitting] refactor: remove even splitting policy #6500

p0mvn opened this issue Sep 23, 2023 · 0 comments · Fixed by #6512
Assignees
@p0mvn
Labels
F: volume-splitting feat: Volume-splitting incentives

Comments

@p0mvn
Copy link
Member

p0mvn commented Sep 23, 2023 •
edited
Loading

Suggesting to remove even split policy altogether as it seems to have been added as a placeholder before volume splitting policy was implemented

We also decided to keep it out of scope of the initial volume splitting implementation per TDD.

Originally posted by @AlpinYukseloglu in #6431 (comment)

A/C

  • Even volume splitting policy is removed
  • Tests updated where applicable
@github-project-automation github-project-automation bot moved this to Needs Triage 🔍 in Osmosis Chain Development Sep 23, 2023
@p0mvn p0mvn added the F: volume-splitting feat: Volume-splitting incentives label Sep 23, 2023
This was referenced Sep 23, 2023
Closed
Merged
Open
Merged
@p0mvn p0mvn self-assigned this Sep 23, 2023
@github-project-automation github-project-automation bot moved this from Needs Triage 🔍 to Done ✅ in Osmosis Chain Development Sep 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@p0mvn p0mvn

Labels
F: volume-splitting feat: Volume-splitting incentives
Projects
Osmosis Chain Development
Archived in project
Milestone
No milestone
1 participant
@p0mvn