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

[VS Incentives] refactor: unexported addToGaugeRewards function without token transfer #6433

Closed
Tracked by #6057
p0mvn opened this issue Sep 19, 2023 · 0 comments · Fixed by #6445 or #6446
Closed
Tracked by #6057

[VS Incentives] refactor: unexported addToGaugeRewards function without token transfer #6433

p0mvn opened this issue Sep 19, 2023 · 0 comments · Fixed by #6445 or #6446
Assignees
Labels
F: volume-splitting feat: Volume-splitting incentives

Comments

@p0mvn
Copy link
Member

p0mvn commented Sep 19, 2023

Context

AddToGaugeRewards takes in a sender. We use it for distributing incentives from group gauge to underlying gauges.

The problem is that we end up transferring tokens redundantly from module account to itself.

A/C

@p0mvn p0mvn added the F: volume-splitting feat: Volume-splitting incentives label Sep 19, 2023
@github-project-automation github-project-automation bot moved this to Needs Triage 🔍 in Osmosis Chain Development Sep 19, 2023
@p0mvn p0mvn changed the title refactor: unexported addToGaugeRewards function without token transfer [VS Incentives] refactor: unexported addToGaugeRewards function without token transfer Sep 19, 2023
@p0mvn p0mvn self-assigned this Sep 19, 2023
@github-project-automation github-project-automation bot moved this from Needs Triage 🔍 to Done ✅ in Osmosis Chain Development Sep 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
F: volume-splitting feat: Volume-splitting incentives
Projects
Archived in project
1 participant