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

Rename assigner_on_demand to on_demand #4706

Merged
merged 85 commits into from
Jul 22, 2024
Merged

Rename assigner_on_demand to on_demand #4706

merged 85 commits into from
Jul 22, 2024

Conversation

s0me0ne-unkn0wn
Copy link
Contributor

Closes #3425

antonva and others added 30 commits March 21, 2024 10:34
@antonva
Start implementing request revenue info
aabcb4b
@antonva
Add revenue tracking to on demand pallet
d8df101
@antonva
Add revenue call to broker pallet
8f7c179
@antonva
Partial impl of revenue from relay chain
54e0326
@antonva
Move xcm sending from ondemand to coretime pallet
d8bfb72
@antonva
Move ondemand types into separate module
f3dfe44
@antonva
Add pallet id to on demand
4f1a44d
@antonva
Fix tests for on demand
ef1174b
@antonva
Rework get_revenue
21f9657
@antonva
Add empty weight implementations
4d00c85
@antonva
Add revenueinbox for broker pallet
8251ede
@antonva
Merge branch 'master' into ava-request-revenue-info
af93286
@antonva
Add log target to coretime pallet
7dd6a34
@antonva
Fix coretime benchmarking
d5d84e3
@antonva
Remove check notify revenue info
aecc034
@antonva
Update broker pallet for revenue info
1f9a4de
@antonva
Remove negative imbalance shorthand
6879dd7
@antonva
Update revenue info calls
7a1f198
@antonva
Remove unused imports
c7d586d
@antonva
Fix benchmarks
74c88b1
@antonva
Merge branch 'master' into ava-request-revenue-info
ff69a3d
@antonva
Fix tests
aa137da
@antonva
Update revenue_until fn
f255e47
@antonva
Remove unused imports
a6be112
@s0me0ne-unkn0wn
Adress minor discussions
f8da2cd
@s0me0ne-unkn0wn
Merge remote-tracking branch 'origin/master' into ava-request-revenue…
0e99692 
…-info
@s0me0ne-unkn0wn
Fix revenue calculation
2e0e3ef
@s0me0ne-unkn0wn
Propose different implementation of revenue reporting
0f6a2ba
@s0me0ne-unkn0wn
Minor fix
af899e2
@s0me0ne-unkn0wn
Rename call
f1b9000
Base automatically changed from ava-request-revenue-info to master June 26, 2024 17:12
@s0me0ne-unkn0wn s0me0ne-unkn0wn added the T2-pallets This PR/Issue is related to a particular pallet. label Jul 18, 2024
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6742763

@s0me0ne-unkn0wn s0me0ne-unkn0wn marked this pull request as ready for review July 19, 2024 07:47
ggwpez
ggwpez reviewed Jul 22, 2024
View reviewed changes
prdoc/pr_4706.prdoc Outdated Show resolved Hide resolved
@s0me0ne-unkn0wn s0me0ne-unkn0wn added this pull request to the merge queue Jul 22, 2024
Merged via the queue into master with commit f923443 Jul 22, 2024
160 of 162 checks passed
@s0me0ne-unkn0wn s0me0ne-unkn0wn deleted the s0me0ne/rename-assigner-on-demand branch July 22, 2024 14:47
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
@s0me0ne-unkn0wn @antonva @TarekkMA
Rename assigner_on_demand to on_demand (paritytech#4706)
7d51040 
Closes paritytech#3425

---------

Co-authored-by: antonva <[email protected]>
Co-authored-by: command-bot <>
@ggwpez ggwpez mentioned this pull request Sep 9, 2024
Closed
@github-actions github-actions bot mentioned this pull request Oct 8, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ggwpez ggwpez ggwpez approved these changes

@bkchr bkchr bkchr approved these changes

@eskimor eskimor Awaiting requested review from eskimor

@seadanda seadanda Awaiting requested review from seadanda

@sandreim sandreim Awaiting requested review from sandreim

Assignees
No one assigned
Labels
T2-pallets This PR/Issue is related to a particular pallet.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Consider moving the "order on-demand" extrinsics
5 participants
@s0me0ne-unkn0wn @paritytech-cicd-pr @bkchr @ggwpez @antonva