[assets tx payment] configurable default asset

[dzmitry-lahoda]

• Description:

Allows user to configure default tx payment asset. See #12055

• Is there something left for follow-up PRs?
  Seems benchmarking can be improved, so not sure I have examples how to do multi assets in this repo (so it should be tested in cumulus).

cumulus companion:paritytech/cumulus#1572

• Labels: You labeled the PR appropriately if you have permissions to do so:
  • A* for PR status (one required)
  • B* for changelog (one required)
  • C* for release notes (exactly one required)
  • D* for various implications/requirements
  • Github project assignment
• Related Issues: Related #WIP #CU-2nhgxne BYOG pallet version with custom header ComposableFi/composable#1456
• 2 Reviewers: You asked at least two reviewers to review. If you aren't sure, start with GH suggestions.
• Style Guide: Your PR adheres to [the style guide]
• Runtime Version: You bumped the runtime version if there are breaking changes in the runtime. No breaking runtime, only compile time.
• Docs: You updated any rustdocs which may need to change.
• Polkadot Companion: Has the PR altered the external API or interfaces used by Polkadot?
  • If so, do you have the corresponding Polkadot PR ready?
  • Optionally: Do you have a corresponding Cumulus PR?

[ggwpez]

cc @shawntabrizi

[dzmitry-lahoda]

seems like other prs also fail with

     Compiling pallet-contracts v4.0.0-dev (/builds/parity/mirrors/substrate/frame/contracts)
  error: generic arguments must come before the first constraint
     --> /builds/parity/mirrors/substrate/frame/uniques/src/lib.rs:114:4
      |
  113 |             Success = Self::AccountId,
      |             ------------------------- constraint
  114 |             Self::Origin,
      |             ^^^^^^^^^^^^
  115 |             Self::CollectionId,
      |             ^^^^^^^^^^^^^^^^^^ generic arguments
      |
  help: move the constraint after the generic arguments
      |
  112 |         type CreateOrigin: EnsureOriginWithArg<Self::Origin, Self::CollectionId, Success = Self::AccountId>;
      |                                               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  warning: ignoring -C extra-filename flag due to -o flag
  warning: `pallet-uniques` (lib) generated 1 warning
  error: could not compile `pallet-uniques` due to previous error; 1 warning emitted
  warning: build failed, waiting for other jobs to finish...

[ggwpez]

bot rebase

[paritytech-processbot]

Error: Command 'Command { std: "git" "push" "--porcelain" "dzmitry-lahoda-forks" "dz/assets-tx-config", kill_on_drop: false }' failed with status Some(128); output: remote: Permission to dzmitry-lahoda-forks/substrate.git denied to paritytech-processbot[bot].
fatal: unable to access 'https://x-access-token:${SECRET}@github.com/dzmitry-lahoda-forks/substrate.git/': The requested URL returned error: 403

[Polkadot-Forum]

This pull request has been mentioned on Polkadot Forum. There might be relevant details there:

https://forum.polkadot.network/t/parachain-technical-summit-next-steps/51/21

[stale]

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

[dzmitry-lahoda]

i will rebase soon to latest master

[stale]

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

[dzmitry-lahoda]

Fixed merge conflicts and updated to latest master on my branch. Not sure, seems there is no way to reopen PR with stale bot. Only to create new one.

Here is updated branch https://github.com/dzmitry-lahoda-forks/substrate/tree/dz/assets-tx-config (same repo and branch as this PR actually - no changes).