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

πŸ‘¨πŸ»β€πŸ’» Allow polimec receiver pallet to have any index #327

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

JuaniRios
Copy link
Contributor

@JuaniRios JuaniRios commented Jun 19, 2024

What?

  • Allow parachains to have their polimec receiver pallet assigned to any pallet index

Why?

  • We should not opinionate that

How?

  • We no longer have the config item PolimecReceiverInfo.
  • We previously needed it because the PalletInfo items returned on the pallet query were set to private. Now we can just check the pallet name and save the index for later

Testing?

  • Modify penpal so the receiver has a different index and run the integration ct_migration tests

Copy link
Contributor Author

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 small ct migration fixes Allow polimec receiver pallet to have any index Jun 19, 2024
@JuaniRios JuaniRios changed the title Allow polimec receiver pallet to have any index πŸ‘¨πŸ»β€πŸ’» Allow polimec receiver pallet to have any index Jun 19, 2024
@JuaniRios JuaniRios marked this pull request as ready for review June 19, 2024 13:12
Copy link

graphite-app bot commented Jun 19, 2024

Graphite Automations

"Auto-assign PRs to author" took an action on this PR β€’ (06/19/24)

1 assignee was added to this PR based on Juan Ignacio Rios's automation.

@JuaniRios JuaniRios merged commit 205b7c4 into main Jun 20, 2024
@JuaniRios JuaniRios deleted the 06-18-small_ct_migration_fixes branch June 20, 2024 11:44
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.

2 participants