Skip to content

[18.0] [IMP] spreadsheet_oca: email digest subscriptions for spreadsheets - #103

Open
dnplkndll wants to merge 3 commits into
OCA:18.0from
ledoent:18.0-feat-subscriptions
Open

[18.0] [IMP] spreadsheet_oca: email digest subscriptions for spreadsheets#103
dnplkndll wants to merge 3 commits into
OCA:18.0from
ledoent:18.0-feat-subscriptions

Conversation

@dnplkndll

@dnplkndll dnplkndll commented Mar 13, 2026

Copy link
Copy Markdown

Adds email digest subscriptions: send a spreadsheet's live pivot summary to a
partner on a schedule.

Depends on #99 — its two commits are shared and drop out once
that merges.

Non-mechanical points worth flagging

  • Each digest is computed as its own Run As user (with_user). This one
    matters more than elsewhere in the series because the result is emailed: a
    shared cron alone would compute every digest with the install user's rights
    and send it out.
  • The mail carries an explicit email_from. Without one, delivery depends
    on mail.catchall.domain / mail.default.from being configured, and
    mail.mail logs the failure rather than raising it — so digests would go
    silently missing on a stock server.
  • Pivot data is recomputed at send time, not read from the last browser save.

AI-assisted (Claude Code); every change reviewed, tested, and owned by the author.

@dnplkndll dnplkndll changed the title [IMP] spreadsheet_oca: email digest subscriptions for spreadsheets [18.0] [IMP] spreadsheet_oca: email digest subscriptions for spreadsheets Mar 13, 2026
@dnplkndll
dnplkndll force-pushed the 18.0-feat-subscriptions branch from f340899 to e95f7db Compare March 13, 2026 02:05
@dnplkndll
dnplkndll force-pushed the 18.0-feat-subscriptions branch from e95f7db to fbff434 Compare May 13, 2026 20:39
@dnplkndll
dnplkndll force-pushed the 18.0-feat-subscriptions branch from fbff434 to 8515400 Compare July 27, 2026 16:25
@dnplkndll
dnplkndll force-pushed the 18.0-feat-subscriptions branch 2 times, most recently from 2bf6339 to a32675b Compare July 27, 2026 18:15
@dnplkndll
dnplkndll force-pushed the 18.0-feat-subscriptions branch from a32675b to 61087ad Compare July 27, 2026 18:42
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.

2 participants