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

fix(pallet-storage-provider): remove Policy #591

Merged
merged 2 commits into from
Nov 20, 2024
Merged

Conversation

jmg-duarte
Copy link
Collaborator

Description

Closes #455

Replaces the Policy structure with more straight forward dependency on the constants defined by the Config.

Details in #455 (comment)

Checklist

  • Make sure that you described what this change does.
  • Have you tested this solution?
  • Were there any alternative implementations considered?

@jmg-duarte jmg-duarte added enhancement New feature or request pallet-storage-provider Relates to the Storage Provider Pallet labels Nov 19, 2024
@jmg-duarte jmg-duarte added this to the Phase 3 milestone Nov 19, 2024
@jmg-duarte jmg-duarte self-assigned this Nov 19, 2024
@jmg-duarte jmg-duarte added the ready for review Review is needed label Nov 20, 2024
@jmg-duarte jmg-duarte enabled auto-merge (squash) November 20, 2024 11:49
@jmg-duarte jmg-duarte merged commit 339107f into develop Nov 20, 2024
5 checks passed
@jmg-duarte jmg-duarte deleted the feat/455/policy branch November 20, 2024 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pallet-storage-provider Relates to the Storage Provider Pallet ready for review Review is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SP pallet: Review Policy type
4 participants