Skip to content

feat(newsletter): fix immediate-send being counted in newsletter rate…#150

Merged
satwikkansal merged 2 commits intoknowsletter-brandingfrom
feature/newsletter-scheduling-ux-improvements
Feb 11, 2026
Merged

feat(newsletter): fix immediate-send being counted in newsletter rate…#150
satwikkansal merged 2 commits intoknowsletter-brandingfrom
feature/newsletter-scheduling-ux-improvements

Conversation

@BhaveshKukreja29
Copy link
Collaborator

Closes #149 #142 #141

  • mxgo/crud.py: added count_recurring_tasks_for_user to distinguish recurring from one-time tasks.
  • mxgo/schemas.py: added language field to newsletter request.
  • mxgo/api.py: refactored immediate sends to skip scheduling, implemented LLM subject generation, and added date context to research prompts.
  • tests/test_api.py: updated mocks and assertions to match new task-counting logic and verification methods.

Checklist

  • I have created an issue for this change (not mandatory for small changes)
  • My changes are to-the-point
  • Code is styled as the rest of the codebase and linting passes
  • I have tested my changes locally and they work as expected
  • I have updated the documentation (if needed)
  • I have reviewed the code myself once and I don't see any

…limit, add dynamic subjects & language support
@BhaveshKukreja29 BhaveshKukreja29 changed the base branch from master to knowsletter-branding February 5, 2026 10:56
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 5, 2026

Deploying mxtoai-backend with  Cloudflare Pages  Cloudflare Pages

Latest commit: 38b7111
Status: ✅  Deploy successful!
Preview URL: https://65086bc3.mxtoai-backend.pages.dev
Branch Preview URL: https://feature-newsletter-schedulin.mxtoai-backend.pages.dev

View logs

@BhaveshKukreja29 BhaveshKukreja29 marked this pull request as ready for review February 5, 2026 11:12
@satwikkansal satwikkansal merged commit 913391e into knowsletter-branding Feb 11, 2026
1 check passed
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.

add optional language field

2 participants