Skip to content

Weekly emails receipt history#479

Merged
epipav merged 10 commits into
mainfrom
bugfix/weekly-emails-cubejs-problems
Feb 2, 2023
Merged

Weekly emails receipt history#479
epipav merged 10 commits into
mainfrom
bugfix/weekly-emails-cubejs-problems

Conversation

@epipav
Copy link
Copy Markdown
Collaborator

@epipav epipav commented Jan 31, 2023

Changes proposed ✍️

  • Cubejs queries were not hitting the cache because we were sending the custom date ranges in seconds granularity.
  • Now we're sending the queries with day granularity (without the hour part) so that cubejs catches these queries with daily-gran preaggregation tables
  • Introducing weeklyAnalyticsEmailsHistory table, where we keep all email receipts for weekly analytics. We keep track of emailSentAt emailSentTo and weekOfYear.
  • Introducing a convenience script that makes it easier for us to send a weekly email to a specific tenant. This script also checks if the email is already sent, and if it's it won't send the email.
  • After merging this, I'll run the convenience script for tenants that didn't receive an email this week.

Checklist ✅

  • Label appropriately with Feature, Enhancement, or Bug.
  • Tests are passing.
  • New backend functionality has been unit-tested.
  • Environment variables have been updated:
    • Local frontend configuration: frontend/.env.dist.local, frontend/.env.dist.composed.
    • Local backend: backend/.env.dist.local, backend/.env.dist.composed.
    • Configuration docs have been updated.
    • Team members only: update environment variables in override, staging and production env. files and trigger update config script.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.
  • All changes have been tested in a staging site.
  • All changes are working locally running crowd.dev's Docker local environment.

@epipav epipav self-assigned this Jan 31, 2023
@epipav epipav added the Bug Created by Linear-GitHub Sync label Jan 31, 2023
@epipav epipav requested a review from themarolt January 31, 2023 16:29
@epipav epipav marked this pull request as ready for review January 31, 2023 16:30
@epipav epipav merged commit 3691aba into main Feb 2, 2023
@epipav epipav deleted the bugfix/weekly-emails-cubejs-problems branch February 2, 2023 10:09
joanreyero pushed a commit that referenced this pull request Apr 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants