fix: simplify domain verification emails (#376)
This commit is contained in:
@@ -11,7 +11,6 @@ interface DomainVerificationStatusEmailProps {
|
|||||||
domainName: string;
|
domainName: string;
|
||||||
currentStatus: DomainStatus;
|
currentStatus: DomainStatus;
|
||||||
previousStatus: DomainStatus;
|
previousStatus: DomainStatus;
|
||||||
verificationError?: string | null;
|
|
||||||
domainUrl: string;
|
domainUrl: string;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -37,7 +36,6 @@ export function DomainVerificationStatusEmail({
|
|||||||
domainName,
|
domainName,
|
||||||
currentStatus,
|
currentStatus,
|
||||||
previousStatus,
|
previousStatus,
|
||||||
verificationError,
|
|
||||||
domainUrl,
|
domainUrl,
|
||||||
}: DomainVerificationStatusEmailProps) {
|
}: DomainVerificationStatusEmailProps) {
|
||||||
const isSuccess = currentStatus === DomainStatus.SUCCESS;
|
const isSuccess = currentStatus === DomainStatus.SUCCESS;
|
||||||
@@ -89,29 +87,6 @@ export function DomainVerificationStatusEmail({
|
|||||||
</Text>
|
</Text>
|
||||||
)}
|
)}
|
||||||
|
|
||||||
{verificationError ? (
|
|
||||||
<Container
|
|
||||||
style={{
|
|
||||||
backgroundColor: "#fef2f2",
|
|
||||||
border: "1px solid #fecaca",
|
|
||||||
padding: "12px 16px",
|
|
||||||
margin: "0 0 24px 0",
|
|
||||||
borderRadius: "4px",
|
|
||||||
}}
|
|
||||||
>
|
|
||||||
<Text
|
|
||||||
style={{
|
|
||||||
margin: 0,
|
|
||||||
color: "#991b1b",
|
|
||||||
fontSize: 14,
|
|
||||||
textAlign: "left" as const,
|
|
||||||
}}
|
|
||||||
>
|
|
||||||
Verification error: {verificationError}
|
|
||||||
</Text>
|
|
||||||
</Container>
|
|
||||||
) : null}
|
|
||||||
|
|
||||||
<Text
|
<Text
|
||||||
style={{
|
style={{
|
||||||
fontSize: "14px",
|
fontSize: "14px",
|
||||||
|
|||||||
@@ -243,11 +243,9 @@ function shouldSendDomainStatusNotification({
|
|||||||
async function sendDomainStatusNotification({
|
async function sendDomainStatusNotification({
|
||||||
domain,
|
domain,
|
||||||
previousStatus,
|
previousStatus,
|
||||||
verificationError,
|
|
||||||
}: {
|
}: {
|
||||||
domain: Domain;
|
domain: Domain;
|
||||||
previousStatus: DomainStatus;
|
previousStatus: DomainStatus;
|
||||||
verificationError: string | null;
|
|
||||||
}) {
|
}) {
|
||||||
const recipients = (
|
const recipients = (
|
||||||
await db.teamUser.findMany({
|
await db.teamUser.findMany({
|
||||||
@@ -282,7 +280,6 @@ async function sendDomainStatusNotification({
|
|||||||
domainName: domain.name,
|
domainName: domain.name,
|
||||||
currentStatus: domain.status,
|
currentStatus: domain.status,
|
||||||
previousStatus,
|
previousStatus,
|
||||||
verificationError,
|
|
||||||
domainUrl,
|
domainUrl,
|
||||||
});
|
});
|
||||||
const statusMessage =
|
const statusMessage =
|
||||||
@@ -293,7 +290,6 @@ async function sendDomainStatusNotification({
|
|||||||
"Hey,",
|
"Hey,",
|
||||||
null,
|
null,
|
||||||
statusMessage,
|
statusMessage,
|
||||||
verificationError ? `Verification error: ${verificationError}` : null,
|
|
||||||
null,
|
null,
|
||||||
`Open domain settings: ${domainUrl}`,
|
`Open domain settings: ${domainUrl}`,
|
||||||
null,
|
null,
|
||||||
@@ -554,7 +550,6 @@ export async function refreshDomainVerification(
|
|||||||
await sendDomainStatusNotification({
|
await sendDomainStatusNotification({
|
||||||
domain: updatedDomain,
|
domain: updatedDomain,
|
||||||
previousStatus,
|
previousStatus,
|
||||||
verificationError,
|
|
||||||
});
|
});
|
||||||
await setLastNotifiedDomainStatus(domain.id, updatedDomain.status);
|
await setLastNotifiedDomainStatus(domain.id, updatedDomain.status);
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
|
|||||||
Reference in New Issue
Block a user