Skip to content

Commit 931540f

Browse files
feat: address linter feedback for "notifications" emails (#1974)
Co-authored-by: gabriel miranda <[email protected]>
1 parent cbeee98 commit 931540f

File tree

5 files changed

+19
-12
lines changed

5 files changed

+19
-12
lines changed

apps/demo/emails/notifications/papermark-year-in-review.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ export default function PapermarkYearInReviewEmail({
6363
you&apos;ve used Papermark to share your important documents.
6464
</Text>
6565
<Link
66-
href="#"
66+
href="https://www.papermark.com"
6767
className="inline-flex items-center rounded-full bg-gray-900 px-12 py-4 text-center text-sm font-bold text-white no-underline"
6868
>
6969
Share your stats
@@ -197,13 +197,13 @@ export default function PapermarkYearInReviewEmail({
197197
Happy Holidays from the Papermark team :)
198198
</Text>
199199
<Link
200-
href="#"
200+
href="https://www.papermark.com"
201201
className="mt-4 inline-flex items-center rounded-full bg-gray-900 px-12 py-4 text-center text-sm font-bold text-white no-underline"
202202
>
203203
Share your stats
204204
</Link>
205205
<Link
206-
href="https://app.papermark.io/documents"
206+
href="https://www.papermark.com"
207207
className="mt-4 block items-center text-center text-sm font-bold text-gray-900 no-underline"
208208
>
209209
Go to your dashboard

apps/demo/emails/notifications/vercel-invite-user.tsx

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ export const VercelInviteUserEmail = ({
5757
src={`${baseUrl}/static/vercel-logo.png`}
5858
width="40"
5959
height="37"
60-
alt="Vercel"
60+
alt="Vercel Logo"
6161
className="my-0 mx-auto"
6262
/>
6363
</Section>
@@ -86,14 +86,15 @@ export const VercelInviteUserEmail = ({
8686
src={userImage}
8787
width="64"
8888
height="64"
89+
alt={`${username}'s profile picture`}
8990
/>
9091
</Column>
9192
<Column align="center">
9293
<Img
9394
src={`${baseUrl}/static/vercel-arrow.png`}
9495
width="12"
9596
height="9"
96-
alt="invited you to"
97+
alt="Arrow indicating invitation"
9798
/>
9899
</Column>
99100
<Column align="left">
@@ -102,6 +103,7 @@ export const VercelInviteUserEmail = ({
102103
src={teamImage}
103104
width="64"
104105
height="64"
106+
alt={`${teamName} team logo`}
105107
/>
106108
</Column>
107109
</Row>
@@ -145,7 +147,7 @@ VercelInviteUserEmail.PreviewProps = {
145147
invitedByEmail: '[email protected]',
146148
teamName: 'Enigma',
147149
teamImage: `${baseUrl}/static/vercel-team.png`,
148-
inviteLink: 'https://vercel.com/teams/invite/foo',
150+
inviteLink: 'https://vercel.com',
149151
inviteFromIp: '204.13.186.218',
150152
inviteFromLocation: 'São Paulo, Brazil',
151153
} as VercelInviteUserEmailProps;

apps/demo/emails/notifications/yelp-recent-login.tsx

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ export const YelpRecentLoginEmail = ({
4444
<Preview>Yelp recent login</Preview>
4545
<Container>
4646
<Section style={logo}>
47-
<Img src={`${baseUrl}/static/yelp-logo.png`} />
47+
<Img src={`${baseUrl}/static/yelp-logo.png`} alt="Yelp logo" />
4848
</Section>
4949

5050
<Section style={content}>
@@ -53,6 +53,7 @@ export const YelpRecentLoginEmail = ({
5353
style={image}
5454
width={620}
5555
src={`${baseUrl}/static/yelp-header.png`}
56+
alt="Yelp header illustration"
5657
/>
5758
</Row>
5859

@@ -122,6 +123,7 @@ export const YelpRecentLoginEmail = ({
122123
style={image}
123124
width={620}
124125
src={`${baseUrl}/static/yelp-footer.png`}
126+
alt="Yelp footer decoration"
125127
/>
126128
</Section>
127129

@@ -166,7 +168,7 @@ const logo = {
166168
};
167169

168170
const buttonContainer = {
169-
display: 'flex',
171+
textAlign: 'center' as const,
170172
};
171173

172174
const button = {
@@ -176,8 +178,9 @@ const button = {
176178
fontWeight: 'bold',
177179
border: '1px solid rgb(0,0,0, 0.1)',
178180
cursor: 'pointer',
179-
margin: '0 auto',
181+
display: 'inline-block',
180182
padding: '12px 30px',
183+
textDecoration: 'none',
181184
};
182185

183186
const content = {

packages/react-email/src/cli/commands/testing/__snapshots__/export.spec.ts.snap

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ exports[`email export 1`] = `
4242
<tr>
4343
<td>
4444
<img
45-
alt="Vercel"
45+
alt="Vercel Logo"
4646
height="37"
4747
src="/static/vercel-logo.png"
4848
style="margin-top:0px;margin-bottom:0px;margin-left:auto;margin-right:auto;display:block;outline:none;border:none;text-decoration:none"
@@ -90,20 +90,22 @@ exports[`email export 1`] = `
9090
<tr style="width:100%">
9191
<td align="right" data-id="__react-email-column">
9292
<img
93+
alt="undefined&#x27;s profile picture"
9394
height="64"
9495
style="border-radius:9999px;display:block;outline:none;border:none;text-decoration:none"
9596
width="64" />
9697
</td>
9798
<td align="center" data-id="__react-email-column">
9899
<img
99-
alt="invited you to"
100+
alt="Arrow indicating invitation"
100101
height="9"
101102
src="/static/vercel-arrow.png"
102103
style="display:block;outline:none;border:none;text-decoration:none"
103104
width="12" />
104105
</td>
105106
<td align="left" data-id="__react-email-column">
106107
<img
108+
alt="undefined team logo"
107109
height="64"
108110
style="border-radius:9999px;display:block;outline:none;border:none;text-decoration:none"
109111
width="64" />

0 commit comments

Comments
 (0)