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

Remove billing support email #6143

Merged
merged 1 commit into from
Oct 23, 2023
Merged

Conversation

jingcheng16
Copy link
Contributor

@jingcheng16 jingcheng16 commented Oct 11, 2023

Ticket: https://dimagi-dev.atlassian.net/browse/SAAS-14988

Environments Affected

All env will be affected: Staging, India, Swiss, Production

This is already deployed to staging.

I will run update-config, and restart service, to india, swiss and production, after this PR is approved and merged.

@@ -78,7 +78,7 @@ data_email: [email protected]
subscription_change_email: [email protected]
internal_subscription_change_email: [email protected]
billing_email: [email protected]
invoicing_contact_email: droberts+billing-support@dimagi.com
invoicing_contact_email: droberts+accounts@dimagi.com
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI @dannyroberts, you may need to update mail filters for this.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. But also, eh, this "environment" is defunct anyway. Can probably just remove it entirely.

@gherceg
Copy link
Contributor

gherceg commented Oct 12, 2023

I will run update-config, and restart service, to india, swiss and production

You may already know this but honestly this is also to clarify my own understanding - I think service commcare restart will need to be scheduled during maintenance hours for production (via the maintenance feed). I believe it is true that this command doesn't disrupt services since it is a rolling restart that doesn't impact formplayer machines, but as an extra precaution we still want to schedule it during maintenance hours. @dannyroberts or @millerdev does that sound right to you?

@dannyroberts
Copy link
Member

I'll offline with you @gherceg about rollout procedures

@jingcheng16 jingcheng16 merged commit 5d1c11e into master Oct 23, 2023
2 checks passed
@jingcheng16 jingcheng16 deleted the jc/remove-billing-support-email branch April 30, 2024 15:26
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.

4 participants