Skip to content

Add check for weekly emails retry#1007

Merged
garrrikkotua merged 1 commit intofeature/segmentsfrom
fix/check-weekly-emails
Jun 20, 2023
Merged

Add check for weekly emails retry#1007
garrrikkotua merged 1 commit intofeature/segmentsfrom
fix/check-weekly-emails

Conversation

@garrrikkotua
Copy link
Copy Markdown
Contributor

@garrrikkotua garrrikkotua commented Jun 20, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 6bfa723

Added a feature flag for weekly analytics emails. The weeklyAnalyticsEmailsWorker.ts file now respects the flag and skips sending emails if disabled.

🤖 Generated by Copilot at 6bfa723

Weekly analytics emails are the tools of the oppressor
They track your every move and make you feel lesser
But we have the power to turn off the feature
With a simple configuration flag we can resist the creature

Why

How

🤖 Generated by Copilot at 6bfa723

  • Add a feature flag to enable or disable weekly analytics emails (link, link)
  • Import WEEKLY_EMAILS_CONFIG from config.ts to access the flag value (link)
  • Check WEEKLY_EMAILS_CONFIG.enabled before sending emails and return a response if disabled (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@garrrikkotua garrrikkotua merged commit 980263d into feature/segments Jun 20, 2023
@garrrikkotua garrrikkotua deleted the fix/check-weekly-emails branch June 20, 2023 13:32
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.

1 participant