Skip to content

Conversation

@cmraible
Copy link
Collaborator

@cmraible cmraible commented Jan 27, 2026

ref https://linear.app/ghost/issue/NY-964/cleanup-remove-welcomeemailsendinstantly-configuration

Initially the process outbox job would run at most every 5 minutes, so we created this memberWelcomeEmailSendInstantly configuration parameter to run the job much more frequently for the sake of our e2e tests. We've since updated the implementation so the process-outbox job is triggered immediately when a member signs up anyways, so this configuration parameter is now redundant and unnecessary.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 27, 2026

Walkthrough

This change removes the memberWelcomeEmailSendInstantly configuration property from the Ghost codebase. The property is eliminated from the test fixture configuration in e2e/helpers/playwright/fixture.ts and the outbox job scheduling logic in ghost/core/core/server/services/outbox/jobs/index.js. The removal simplifies the cron schedule computation by eliminating the conditional branch that previously toggled behavior based on this configuration. Documentation and example usage are updated to reflect these changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: removing the memberWelcomeEmailSendInstantly configuration parameter, which is the primary focus of both modified files.
Description check ✅ Passed The description explains the purpose and rationale for the change, relating directly to the removal of the memberWelcomeEmailSendInstantly parameter across the codebase.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cmraible cmraible changed the title Removed memberWelcomeEmailSendInstantly configuration parameter Jan 27, 2026
@cmraible cmraible force-pushed the chris-ny-964-cleanup-remove-welcomeemailsendinstantly-configuration branch from b4af01f to 4300df4 Compare January 28, 2026 00:34
@cmraible cmraible marked this pull request as ready for review January 28, 2026 00:56
@cmraible cmraible closed this Jan 28, 2026
@cmraible
Copy link
Collaborator Author

Still planning to do this, but this isn't the right way to do it 😅

@cmraible cmraible reopened this Jan 29, 2026
@cmraible cmraible requested review from troyciesco and removed request for ibalosh January 29, 2026 18:06
@cmraible cmraible force-pushed the chris-ny-964-cleanup-remove-welcomeemailsendinstantly-configuration branch from ddd2b1d to 29886d8 Compare January 29, 2026 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants