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

🤝 One Token Model - Pallet Funding Integration #401

Open
wants to merge 1 commit into
base: 09-10-one-token-model
Choose a base branch
from
Open

🤝 One Token Model - Pallet Funding Integration #401

wants to merge 1 commit into from
+697 −300

Conversation

JuaniRios
Copy link
Contributor

@JuaniRios JuaniRios commented Sep 18, 2024 •
edited
Loading

What?

Integrate the OTM flow in pallet-funding

How?

  • Tightly couple pallet-proxy-bonding with pallet-funding
  • Abstract multiplier parameter in extrinsic to an enum called ParticipationMode with variants Classic(multiplier) and OTM
  • Abstract the plmc bonding to a new function that takes the participation mode, and either does the classic bonding, or calls the proxy-bonding logic
  • On settlement start, set the release type to Refunded or Locked in proxy-bonding pallet.
  • On participation settlements, if the project failed, or a bid has some refund, call the refund_fee of proxy-bonding

Testing?

In the next PR

@JuaniRios Graphite App
Copy link
Contributor Author

JuaniRios commented Sep 18, 2024 •
edited
Loading

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @JuaniRios and the rest of your teammates on Graphite Graphite

@JuaniRios JuaniRios mentioned this pull request Sep 18, 2024
Open
@JuaniRios JuaniRios changed the title pallet funding OTM integration 🤝 OTM integration Sep 18, 2024
@JuaniRios JuaniRios changed the title 🤝 OTM integration 🤝 OTM funding integration Sep 18, 2024
@JuaniRios JuaniRios force-pushed the 09-18-pallet_funding_otm_integration branch 3 times, most recently from 3849071 to 59622fb Compare September 18, 2024 14:14
@JuaniRios JuaniRios mentioned this pull request Sep 19, 2024
Open
@JuaniRios JuaniRios changed the title 🤝 OTM funding integration 🤝 One Token Model - pallet-funding integration Sep 19, 2024
@JuaniRios JuaniRios changed the title 🤝 One Token Model - pallet-funding integration 🤝 One Token Model - Pallet Funding Integration Sep 19, 2024
@JuaniRios JuaniRios changed the title 🤝 One Token Model - Pallet Funding Integration 🤝 One Token Model - Pallet Funding Integration Sep 19, 2024
@JuaniRios JuaniRios force-pushed the 09-18-pallet_funding_otm_integration branch from 59622fb to 7b27337 Compare September 20, 2024 12:04
@JuaniRios JuaniRios force-pushed the 09-18-pallet_funding_otm_integration branch 4 times, most recently from c96cdf6 to 0cce953 Compare September 23, 2024 10:49
@JuaniRios JuaniRios force-pushed the 09-18-pallet_funding_otm_integration branch from 0cce953 to a04bf46 Compare September 23, 2024 10:51
@JuaniRios JuaniRios self-assigned this Sep 24, 2024
@JuaniRios JuaniRios marked this pull request as ready for review September 24, 2024 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lrazovic lrazovic Awaiting requested review from lrazovic

@vstam1 vstam1 Awaiting requested review from vstam1

Assignees

@JuaniRios JuaniRios

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JuaniRios