Skip to content

chore(email-link-delete): Disable email if no opens after 7 days inst…#2250

Merged
evanhutnik merged 1 commit intomainfrom
evan/first-open-disable
Mar 30, 2026
Merged

chore(email-link-delete): Disable email if no opens after 7 days inst…#2250
evanhutnik merged 1 commit intomainfrom
evan/first-open-disable

Conversation

@evanhutnik
Copy link
Copy Markdown
Contributor

…ead of 14

@evanhutnik evanhutnik requested a review from a team as a code owner March 30, 2026 14:38
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 30, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: b65a80c9-64be-4d70-a2c2-e1164ed5ee35

📥 Commits

Reviewing files that changed from the base of the PR and between 28e7462 and e341c66.

📒 Files selected for processing (1)
  • infra/stacks/email-service/Pulumi.prod.yaml

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated email service production configuration to reduce the retention period for unused email items from 14 days to 7 days, enabling faster cleanup and improved storage management.

Walkthrough

Updated the Pulumi production configuration for the email service stack, modifying the delete_unused_after_days parameter from 14 to 7 days. This changes the retention period for unused resources in the production environment.

Changes

Cohort / File(s) Summary
Pulumi Configuration
infra/stacks/email-service/Pulumi.prod.yaml
Modified email-service:delete_unused_after_days configuration value from 14 to 7 days.
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the conventional commits format with 'chore:' prefix and is 70 characters, meeting all specified requirements.
Description check ✅ Passed The description accurately relates to the changeset, explaining the core change of reducing the email deletion threshold from 14 to 7 days.

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


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

@github-actions github-actions bot added the infra label Mar 30, 2026
@evanhutnik evanhutnik merged commit 7b38fc7 into main Mar 30, 2026
22 checks passed
@evanhutnik evanhutnik deleted the evan/first-open-disable branch March 30, 2026 14:41
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.

1 participant