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

[FRAME] MQ processor should be transactional #2441

Open
ggwpez opened this issue Nov 22, 2023 · 3 comments
Open

[FRAME] MQ processor should be transactional #2441

ggwpez opened this issue Nov 22, 2023 · 3 comments
Assignees
@lean-apple
Labels
C1-mentor A task where a mentor is available. Please indicate in the issue who the mentor could be. D1-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase. I5-enhancement An additional feature request. T2-pallets This PR/Issue is related to a particular pallet.

Comments

@ggwpez
Copy link
Member

ggwpez commented Nov 22, 2023

Follow up to #2356 (comment) where it was noticed that the MQ pallet does not enforce transactional processing.
Should probably be fixed, but it a breaking change, therefore creating a new issue for it.
@ggwpez ggwpez changed the title [FRAME] MQ message processor should be transactional [FRAME] MQ processor should be transactional Nov 22, 2023
@ggwpez ggwpez mentioned this issue Nov 22, 2023
Merged
@ggwpez ggwpez added C1-mentor A task where a mentor is available. Please indicate in the issue who the mentor could be. I5-enhancement An additional feature request. T2-pallets This PR/Issue is related to a particular pallet. D1-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase. labels Feb 5, 2024
@asoliman92
Copy link

I'd like to pick this up please @ggwpez

@lean-apple
Copy link
Contributor

Is it in progress or I can try also ?

@ggwpez
Copy link
Member Author

ggwpez commented Apr 9, 2024

Yea seems to be stale, going to re-assign. Thanks @lean-apple!

@ggwpez ggwpez assigned lean-apple and unassigned asoliman92 Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lean-apple lean-apple

Labels
C1-mentor A task where a mentor is available. Please indicate in the issue who the mentor could be. D1-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase. I5-enhancement An additional feature request. T2-pallets This PR/Issue is related to a particular pallet.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@asoliman92 @ggwpez @lean-apple