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

[AHM] Pallet Indices #577

Merged
merged 5 commits into from
Feb 6, 2025
Merged

[AHM] Pallet Indices #577

merged 5 commits into from
Feb 6, 2025

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Feb 4, 2025

To be merged into the AHM working branch

  • Does not require a CHANGELOG entry

Signed-off-by: Oliver Tale-Yazdi <[email protected]>

pallet_indices::Accounts::<T>::insert(
&index.index,
(index.who, index.deposit, index.frozen),
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could think about unlocking some of the deposit, but that would make verification more difficult. Will leave it as is now.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably should start creating issues for Post AHM project

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am using this for now paritytech/polkadot-sdk#7369 which we can then split up into real issues.

@ggwpez ggwpez marked this pull request as ready for review February 4, 2025 21:30
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@ggwpez ggwpez changed the title indices [AHM] Pallet Indices Feb 4, 2025
@ggwpez ggwpez mentioned this pull request Feb 3, 2025
64 tasks
system-parachains/asset-hubs/asset-hub-polkadot/src/lib.rs Outdated Show resolved Hide resolved

pallet_indices::Accounts::<T>::insert(
&index.index,
(index.who, index.deposit, index.frozen),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably should start creating issues for Post AHM project

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@ggwpez ggwpez merged commit 88738f2 into dev-asset-hub-migration Feb 6, 2025
30 of 47 checks passed
@ggwpez ggwpez deleted the oty-indices branch February 6, 2025 13:53
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