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

🧪 One Token Model - Pallet Funding Tests #402

Draft
wants to merge 1 commit into
base: 09-18-pallet_funding_otm_integration
Choose a base branch
from

Conversation

JuaniRios
Copy link
Contributor

@JuaniRios JuaniRios commented Sep 19, 2024

What?

  • Change mock AccountId from u32 to u64
  • Test the OTM flow inside pallet-funding

Why?

  • PalletId account derivations start with a 32bit constant, so that meant the proxy-bonding escrow and the project_id escrow had the same account
  • So far we only tested the proxy-bonding in a vacuum.

How?

Testing?

Screenshots (optional)

Anything Else?

Copy link
Contributor Author

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 changed the title otm_funding_tests 🧪 OTM Funding tests Sep 19, 2024
@JuaniRios JuaniRios changed the title 🧪 OTM Funding tests 🧪One Token Model - pallet-funding tests Sep 19, 2024
@JuaniRios JuaniRios changed the title 🧪One Token Model - pallet-funding tests 🧪One Token Model - Pallet Funding Tests Sep 19, 2024
@JuaniRios JuaniRios changed the title 🧪One Token Model - Pallet Funding Tests 🧪 One Token Model - Pallet Funding Tests 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 from 7b27337 to ae97cae Compare September 20, 2024 13:20
@JuaniRios JuaniRios force-pushed the 09-18-pallet_funding_otm_integration branch from ae97cae to b9e0a82 Compare September 20, 2024 13:23
@JuaniRios JuaniRios force-pushed the 09-18-pallet_funding_otm_integration branch from b9e0a82 to c96cdf6 Compare September 23, 2024 09:23
@JuaniRios JuaniRios force-pushed the 09-18-pallet_funding_otm_integration branch 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
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