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

Control sending weekly user research email via environment variable #4377

Merged
merged 1 commit into from
Feb 10, 2025

Conversation

klssmith
Copy link
Contributor

@klssmith klssmith commented Feb 6, 2025

We were always sending the weekly user research email, regardless of environment. We had decided at the time that this was fine, but now that we may have long-lived environments set up for user research we should avoid sending the email to any of the participants if we forget to archive them after the session.

This adds an environment variable to control whether or not the emails should be sent, which is an approach we take for other tasks we only want to happen in certain environments.

We were always sending the weekly user research email, regardless of
environment. We had decided at the time that this was fine, but now that
we may have long-lived environments set up for user research we should
avoid sending the email to any of the participants if we forget to
archive them after the session.

This adds an environment variable to control whether or not the emails
should be sent, which is an approach we take for other tasks we only
want to happen in certain environments.
@klssmith klssmith merged commit 6adc6fc into main Feb 10, 2025
5 checks passed
@klssmith klssmith deleted the user-research-email-flag branch February 10, 2025 15:33
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.

2 participants