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

🎁 [Feature Request]: Get rid of duplicated code for AccountId20 compatibility #258

Open
2 of 3 tasks
ozgunozerk opened this issue Jul 24, 2024 · 0 comments
Open
2 of 3 tasks

🎁 [Feature Request]: Get rid of duplicated code for AccountId20 compatibility #258

ozgunozerk opened this issue Jul 24, 2024 · 0 comments
Assignees
@ozgunozerk
Labels
enhancement New feature or request

Comments

@ozgunozerk
Copy link
Collaborator

templates

  • Generic Runtime Template
  • Evm Template

What is the feature you would like to see?

We duplicated the upstream code for making the XcmFeeToAccount struct compatible with AccountId20 type in our EVM runtime template.

To fix the issue, 2 PRs submitted:

When the new upstream releases containing these changes are out, we should get rid of the duplicated code in our code-base and make our runtime code more compact.

Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ozgunozerk ozgunozerk

Labels
enhancement New feature or request
Projects
Substrate Runtime Templates
Status: 🗒 Backlog
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ozgunozerk