Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hotfix release 2025.02.03.01 #5588

Closed
wants to merge 1 commit into from
Closed

Hotfix release 2025.02.03.01 #5588

wants to merge 1 commit into from

Conversation

Vinnl
Copy link
Collaborator

@Vinnl Vinnl commented Feb 4, 2025

This is branched off the tag 2025.02.03, with commit 317064f (from #5587) applied on top.

When we're ready to deploy, I can tag a new release 2025.02.03.01 from this branch so we can deploy that, and after merge it back to main (should be a no-op after 2025.02.03.01).

We email people that their subscription is about to expire, then
mark them as having received the email. We check whether we have
marked someone as having received the email when collecting the
list of subscribers to email, which means that we shouldn't use
that list when determining whether someone can renew.

I've updated it to use a similar check, but to skip the check for
`churn_prevention_email_sent_at`.
@Vinnl Vinnl added the Review: µ Code review time: 5 minutes or less label Feb 4, 2025
@Vinnl Vinnl requested a review from flozia February 4, 2025 12:32
@Vinnl Vinnl self-assigned this Feb 4, 2025
Copy link

github-actions bot commented Feb 4, 2025

@Vinnl Vinnl force-pushed the release/2025.02.03.01 branch from 9c187f8 to eaf716b Compare February 4, 2025 13:59
@Vinnl
Copy link
Collaborator Author

Vinnl commented Feb 4, 2025

Closing this because apparently building the image on non-main doesn't work.

@Vinnl Vinnl closed this Feb 4, 2025
@Vinnl Vinnl deleted the release/2025.02.03.01 branch February 4, 2025 14:08
Copy link

github-actions bot commented Feb 4, 2025

Cleanup completed - database 'blurts-server-pr-5588' destroyed, cloud run service 'blurts-server-pr-5588' destroyed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review: µ Code review time: 5 minutes or less
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants