Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Messenger] Add the--all option to themessenger:failed:remove command#51593

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

alexandre-daubois
Copy link
Member

@alexandre-dauboisalexandre-daubois commentedSep 7, 2023•
edited
Loading

QA
Branch?6.4
Bug fix?no
New feature?yes
Deprecations?no
Tickets-
LicenseMIT
Doc PRTodo

We have a development server (which we don't have direct access to the database). As this server serves as a test for our devs, error messages can accumulate in our failure transport. We wanted to use themessenger:failed:remove command to remove them, but unfortunately, we must provide ids individually. This is problematic as we have several hundreds of failed messages.

This PR adds the--all option to the command. This optionmust be used with the--force option (juste likedoctrine:schema:update --force actually) to work. Example output:

$ bin/console messenger:failed:remove --all --force...Failed Message Details======================                                                                                                                         [WARNING] Message does not appear to have been sent to this transport after failing                                                                                                                                                             ------------ -------------------------   Class        App\Message\YourMessage    Message Id   6                         ------------ -------------------------! [NOTE] 4 messages were removed.

As you can see, you can of course still use the--show-messages option jointly.

valtzu, darthf1, and welcoMattic reacted with thumbs up emoji
@alexandre-daubois
Copy link
MemberAuthor

All comments addressed, thanks! 🙂

@alexandre-daubois
Copy link
MemberAuthor

Addressed all your comments Nicolas, thank you!

@alexandre-dauboisalexandre-dauboisforce-pushed theremove-all-failed-msgs branch 2 times, most recently fromf26d36b tob674207CompareSeptember 19, 2023 14:08
@alexandre-dauboisalexandre-dauboisforce-pushed theremove-all-failed-msgs branch 2 times, most recently from7d62d21 to322d892CompareSeptember 21, 2023 06:57
@nicolas-grekas
Copy link
Member

Thank you@alexandre-daubois.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@welcoMatticwelcoMatticwelcoMattic left review comments

@fabpotfabpotfabpot requested changes

@kbondkbondkbond approved these changes

@nicolas-grekasnicolas-grekasnicolas-grekas approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
6.4
Development

Successfully merging this pull request may close these issues.

6 participants
@alexandre-daubois@nicolas-grekas@fabpot@kbond@welcoMattic@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp