Skip to content

docs: correct news summary cron from daily to weekly in README - #91

Merged
ravixalgorithm merged 1 commit into
Open-Dev-Society:mainfrom
kuishou68:cocoon-fix-openstock-weekly-summary
Sep 21, 2026
Merged

ravixalgorithm merged 1 commit into
Open-Dev-Society:mainfrom
kuishou68:cocoon-fix-openstock-weekly-summary

Conversation

@kuishou68

@kuishou68 kuishou68 commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

What

The README describes the Inngest news-summary workflow as a daily cron (0 12 * * *) sending per-user emails, but the implementation is different:

  • lib/inngest/functions.ts defines weekly-news-summary with trigger { event: 'app/send.weekly.news', cron: '0 9 * * 1' } (every Monday at 9 AM), with a // Rename to Weekly comment right above it.
  • The summary is delivered as a Kit broadcast (send-kit-broadcast step), not a per-user email, and no daily 0 12 * * * cron exists anywhere in the codebase.

Change

Two README lines updated to match the code (Features -> "Email & automation" bullet, and the Inngest workflow list):

  • "Daily news summary emails (cron) personalized using user watchlists" -> "Weekly news summary email (cron) sent as a Kit broadcast"
  • "Cron 0 12 * * * -> Daily News Summary per user" -> "Cron 0 9 * * 1 -> Weekly News Summary (every Monday at 9 AM, Kit broadcast)"

Verification

  • Checked lib/inngest/functions.ts: the only news-summary function is weekly-news-summary with cron 0 9 * * 1; no 0 12 * * * cron exists.
  • Diff is docs-only (README, 2 lines), no code touched.

Summary by CodeRabbit

  • Documentation
    • Updated the news summary email documentation to describe a weekly Kit broadcast instead of daily personalized emails.
    • Documented the revised delivery schedule: every Monday at 9 AM.

The Inngest function is weekly-news-summary (lib/inngest/functions.ts),
triggered by cron
@vercel

vercel Bot commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

@kuishou68 is attempting to deploy a commit to the ravixalgorithm's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 4bbd7af1-b319-4b80-b96a-5baed71086b5

📥 Commits

Reviewing files that changed from the base of the PR and between 53553ec and a2e010e.

📒 Files selected for processing (1)
  • README.md

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Walkthrough

README.md now describes the news summary email as a weekly Kit broadcast sent every Monday at 9 AM.

Changes

News Summary Email Documentation

Layer / File(s) Summary
Update news summary schedule documentation
README.md
The feature description and integration details now state that the news summary uses a weekly Kit broadcast with cron schedule 0 9 * * 1.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: ravixalgorithm

Merge Risk: ⚪ Minimal · up to a2e01

The README schedule and Kit broadcast description match the implemented workflow, and the remaining Nodemailer documentation is not contradictory.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the documentation change and the correction from a daily to a weekly news summary cron. It accurately reflects the main change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Create a new PR

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.

@ravixalgorithm
ravixalgorithm merged commit c83129d into Open-Dev-Society:main Sep 21, 2026
1 of 2 checks passed
@ravixalgorithm

Copy link
Copy Markdown
Member

Thanks @kuishou68 for the fix !!!

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.

2 participants