Skip to content

Commit 67cf27f

Browse files
DMARC not consistently denoted by failures (canada-ca#2374)
* DMARC not consistently denoted by failures Most locations marked what control failed, But the DMARC graphs marked which control passed All locations now go by the failure to be consistent. * update po files * fix calls to api * Run npm run extract Co-authored-by: FestiveKyle <kyle.j.s.sullivan@gmail.com>
1 parent 2beee0a commit 67cf27f

8 files changed

Lines changed: 132 additions & 100 deletions

File tree

frontend/src/DmarcReportPage.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -226,13 +226,13 @@ export default function DmarcReportPage({ summaryListResponsiveWidth }) {
226226
moderate: [
227227
{
228228
name: 'passSpfOnly',
229-
displayName: t`Pass Only SPF`,
229+
displayName: t`Fail DKIM`,
230230
},
231231
],
232232
moderateAlt: [
233233
{
234234
name: 'passDkimOnly',
235-
displayName: t`Pass Only DKIM`,
235+
displayName: t`Fail SPF`,
236236
},
237237
],
238238
weak: [

frontend/src/__tests__/DmarcReportPage.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ describe('<DmarcReportPage />', () => {
179179
</ThemeProvider>
180180
</UserStateProvider>,
181181
)
182-
await waitFor(() => getAllByText(/^Pass Only SPF$/i))
182+
await waitFor(() => getAllByText(/^Fail DKIM$/i))
183183
})
184184

185185
it('renders tables', async () => {

frontend/src/__tests__/DmarcReportSummaryGraph.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,6 @@ describe('<DmarcReportTimeGraph />', () => {
2525
</I18nProvider>
2626
</ThemeProvider>,
2727
)
28-
await waitFor(() => getByText(/Pass Only SPF/i))
28+
await waitFor(() => getByText(/Fail DKIM/i))
2929
})
3030
})

frontend/src/fixtures/summaryListData.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -157,12 +157,12 @@ const strengths = {
157157
],
158158
moderate: [
159159
{
160-
name: 'passSpfOnly',
161-
displayName: t`Pass Only SPF`,
160+
name: 'failDkim',
161+
displayName: t`Fail DKIM`,
162162
},
163163
{
164-
name: 'passDkimOnly',
165-
displayName: t`Pass Only DKIM`,
164+
name: 'failSpf',
165+
displayName: t`Fail SPF`,
166166
},
167167
],
168168
weak: [

frontend/src/locales/en.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,9 @@
192192
"Enter your user account's verified email address and we will send you a password reset link.",
193193
'Envelope From': 'Envelope From',
194194
Fail: 'Fail',
195+
'Fail DKIM': 'Fail DKIM',
195196
'Fail DKIM %': 'Fail DKIM %',
197+
'Fail SPF': 'Fail SPF',
196198
'Fail SPF %': 'Fail SPF %',
197199
February: 'February',
198200
'Follow implementation guide': 'Follow implementation guide',
@@ -376,8 +378,6 @@
376378
'Page Size:': 'Page Size:',
377379
'Page {0} of {1}': ['Page ', ['0'], ' of ', ['1']],
378380
Pass: 'Pass',
379-
'Pass Only DKIM': 'Pass Only DKIM',
380-
'Pass Only SPF': 'Pass Only SPF',
381381
'Pass/Fail Ratios by Domain': 'Pass/Fail Ratios by Domain',
382382
Password: 'Password',
383383
'Password Updated': 'Password Updated',

0 commit comments

Comments
 (0)