Skip to content

docs(status): say Todoist honours --pause-notifications for push and email - #617

Merged
lmjabreu merged 6 commits into
mainfrom
lmjabreu/status-pause-notifications-wording
Oct 29, 2026
Merged

lmjabreu merged 6 commits into
mainfrom
lmjabreu/status-pause-notifications-wording

Conversation

@lmjabreu

@lmjabreu lmjabreu commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Overview

Doist/Todoist#30808 now holds back Todoist push and email notifications while a status with --pause-notifications is active (deployed to production on 23 Oct). The help text and skill reference still said "not Todoist yet". This updates them. In the Todoist Android app only the morning and evening pushes are gated, so the text says so.

Reference

Changelog

Update the --pause-notifications help to say Todoist honours it for push and email, and in the Todoist Android app only for the morning and evening pushes.

Test plan

  1. Run td status set --help.
    • The --pause-notifications option and the footer mention Todoist push and email, and the Todoist Android app's morning and evening pushes only
  2. Run grep -n "pause-notifications" skills/todoist-cli/SKILL.md.
    • The paragraph carries the same wording
  3. With a colleague assigning you a task while td status set "Away" --for 10m --pause-notifications is active:
    • You get no push or email on web or iOS, and the task still shows in the in-app feed

Not verified end to end: I confirmed the backend change is in a production deployment and read the Android limit in the backend README, but I have not seen a notification suppressed.

🤖 Generated with Claude Code

lmjabreu and others added 3 commits October 23, 2026 16:34
…email

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… caveat

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@lmjabreu

Copy link
Copy Markdown
Contributor Author

@doistbot /review

@doistbot doistbot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Docs-only update that aligns the --pause-notifications help text, footer, and skill reference with the new backend behavior (Todoist now honors the flag for push and email, with the Android morning/evening caveat). The wording stays consistent across the CLI help, SKILL_CONTENT, and the generated SKILL.md, so the skill-sync check remains green. No inline issues were flagged in the review.

I also left one optional follow-up note in the details below.

Optional follow-up note (1)
  • P3 src/commands/status/index.ts:44: "Comms, and Todoist push and email" reads as a three-item list and omits the "for" used in the footer of this same file and in the SKILL content. Use "Comms, and Todoist for push and email" to keep the wording clear and consistent.

Share Feedback • Review Logs

… help

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@lmjabreu

Copy link
Copy Markdown
Contributor Author

src/commands/status/index.ts:44: "Comms, and Todoist push and email" reads as a three-item list…

Done. Added "for" so it matches the footer and the skill text.

@lmjabreu
lmjabreu marked this pull request as ready for review October 27, 2026 22:25
…-notifications-wording

# Conflicts:
#	skills/todoist-cli/SKILL.md
#	src/lib/skills/content.ts
@lmjabreu

Copy link
Copy Markdown
Contributor Author

@doistbot /review

@doistbot doistbot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This PR updates docs(status): say Todoist honours --pause-notifications for push and email. No issues were flagged in the reviewed diff.

Share Feedback • Review Logs

@lmjabreu
lmjabreu requested a review from gnapse October 29, 2026 14:32
@lmjabreu
lmjabreu merged commit e62af26 into main Oct 29, 2026
6 checks passed
@lmjabreu
lmjabreu deleted the lmjabreu/status-pause-notifications-wording branch October 29, 2026 22:42
@doist-release-bot

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.8.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants