Skip to content

Add documentation for mautic:campaigns:resume-stuck command#709

Open
rohitpavaskar wants to merge 1 commit into
mautic:7.0from
rohitpavaskar:fix_stuck_campaign
Open

Add documentation for mautic:campaigns:resume-stuck command#709
rohitpavaskar wants to merge 1 commit into
mautic:7.0from
rohitpavaskar:fix_stuck_campaign

Conversation

@rohitpavaskar
Copy link
Copy Markdown

Description

Document the new command that resumes execution for contacts stuck in campaign workflows. This command helps recover contacts that got stuck due to exceptions, server issues, or events added after contacts had already progressed past that point.

Related to: mautic/mautic#15310

Linked issue

mautic/mautic#15310

Screenshots or screen recordings

Document the new command that resumes execution for contacts stuck in
campaign workflows. This command helps recover contacts that got stuck
due to exceptions, server issues, or events added after contacts had
already progressed past that point.

Related to: mautic/mautic#15310
Copy link
Copy Markdown
Member

@escopecz escopecz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adiati98 we should probably create the 7.2 branch for these PR, right?

@rohitpavaskar shall we suggest users to also configure this command to run periodically as a cron job? Or only in special cases?

@adiati98
Copy link
Copy Markdown
Contributor

@escopecz if this aims 7.2, we can create the branch. Would it be based on 7.1 or 7.0?

@escopecz
Copy link
Copy Markdown
Member

It should be based on 7.1

@adiati98
Copy link
Copy Markdown
Contributor

@escopecz noted. Can we hold on a bit? I'll try to review and merge as much PRs as we can for branch 7.1 before creating 7.2.

@escopecz
Copy link
Copy Markdown
Member

Sure thing!

@rohitpavaskar
Copy link
Copy Markdown
Author

ommand to run periodically as a cron j

Yes, This needs to be set as cron job but separate job for each campaign.

@promptless-for-oss
Copy link
Copy Markdown

Thanks for documenting the new mautic:campaigns:resume-stuck command! The documentation is comprehensive and well-structured.

I noticed two numerical values that don't match the source code and should be corrected:

  1. Batch limit default: The docs currently say "Defaults to 1000" but the source code shows the default is 100.

  2. Max records per execution: The docs say "5000 records" but the source code shows MAX_ALLOWED_RECORDS_EACH_PROCESS = 500.


🤖 Review by Promptless

@adiati98
Copy link
Copy Markdown
Contributor

@rohitpavaskar can you please check if the review from Promptless here is correct? TIA! ✨

#709 (comment)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants