Use trailing slashes everywhere

This is the most sweeping breaking change in Prettier 3.0.
mozilla · Oct 16, 2023 · b823acc · b823acc
1 parent a29d009
commit b823acc
Show file tree

Hide file tree

Showing 138 changed files with 786 additions and 783 deletions.
diff --git a/src/apiMocks/mockData.ts b/src/apiMocks/mockData.ts
@@ -46,7 +46,7 @@ export type RandomScanResultOptions = Partial<{
  * @returns A single scan result.
  */
 export function createRandomScanResult(
-  options: RandomScanResultOptions = {}
+  options: RandomScanResultOptions = {},
 ): OnerepScanResultRow {
   faker.seed(options.fakerSeed);
   return {
@@ -60,7 +60,7 @@ export function createRandomScanResult(
     status:
       options.status ??
       (faker.helpers.arrayElement(
-        Object.values(RemovalStatusMap)
+        Object.values(RemovalStatusMap),
       ) as RemovalStatus),
     manually_resolved: options.manually_resolved ?? faker.datatype.boolean(),
     addresses: Array.from({ length: 3 }, () => ({
@@ -91,7 +91,7 @@ export type RandomBreachOptions = Partial<{
 
 // TODO: MNTOR-2033 Update this random breach function with new data breach object, and deprecate all BreachMockItems
 export function createRandomBreach(
-  options: RandomBreachOptions = {}
+  options: RandomBreachOptions = {},
 ): SubscriberBreach {
   const dataClassTypes = options.isHighRiskOnly
     ? HighRiskDataTypes

diff --git a/src/app/(nextjs_migration)/(authenticated)/admin/emails/[template]/page.tsx b/src/app/(nextjs_migration)/(authenticated)/admin/emails/[template]/page.tsx
@@ -103,31 +103,31 @@ function getTemplatesData(l10n: ReactLocalization) {
       template: getPreviewTemplate(
         getVerificationDummyData(appConstants.EMAIL_TEST_RECIPIENT, l10n),
         verifyPartial,
-        l10n
+        l10n,
       ),
     },
     [EmailTemplateType.Notification]: {
       label: "Breach notification",
       template: getPreviewTemplate(
         getNotificationDummyData(appConstants.EMAIL_TEST_RECIPIENT, l10n),
         breachAlertEmailPartial,
-        l10n
+        l10n,
       ),
     },
     [EmailTemplateType.Monthly]: {
       label: "Monthly unresolved breaches",
       template: getPreviewTemplate(
         getMonthlyDummyData(appConstants.EMAIL_TEST_RECIPIENT, l10n),
         monthlyUnresolvedEmailPartial,
-        l10n
+        l10n,
       ),
     },
     [EmailTemplateType.SignupReport]: {
       label: "Signup report",
       template: getPreviewTemplate(
         getSignupReportDummyData(appConstants.EMAIL_TEST_RECIPIENT, l10n),
         signupReportEmailPartial,
-        l10n
+        l10n,
       ),
     },
   };
@@ -145,7 +145,7 @@ function getPreviewOptions(currentTemplateKey: string, data: any) {
     >
       ${data[templateKey].label as string}
     </option>
-  `
+  `,
     )
     .join("");
 

diff --git a/src/app/(nextjs_migration)/(authenticated)/admin/send-test-email/route.ts b/src/app/(nextjs_migration)/(authenticated)/admin/send-test-email/route.ts
@@ -31,13 +31,13 @@ export async function POST(req: NextRequest) {
       const emailTemplate = getTemplate(
         getVerificationDummyData(recipient, l10n),
         verifyPartial,
-        l10n
+        l10n,
       );
       await initEmail(process.env.SMTP_URL);
       await sendEmail(
         recipient,
         l10n.getString("email-subject-verify"),
-        emailTemplate
+        emailTemplate,
       );
       break;
     }
@@ -51,13 +51,13 @@ export async function POST(req: NextRequest) {
       const emailTemplate = getTemplate(
         getMonthlyDummyData(AppConstants.EMAIL_TEST_RECIPIENT, l10n),
         monthlyUnresolvedEmailPartial,
-        l10n
+        l10n,
       );
       await initEmail(process.env.SMTP_URL);
       await sendEmail(
         recipient,
         l10n.getString("email-unresolved-heading"),
-        emailTemplate
+        emailTemplate,
       );
       break;
     }
@@ -66,13 +66,13 @@ export async function POST(req: NextRequest) {
       const emailTemplate = getTemplate(
         getSignupReportDummyData(AppConstants.EMAIL_TEST_RECIPIENT, l10n),
         signupReportEmailPartial,
-        l10n
+        l10n,
       );
       await initEmail(process.env.SMTP_URL);
       await sendEmail(
         recipient,
         l10n.getString("email-subject-found-breaches"),
-        emailTemplate
+        emailTemplate,
       );
       break;
     }
@@ -87,7 +87,7 @@ export async function POST(req: NextRequest) {
   if (emailId !== EmailTemplateType.Notification) {
     return NextResponse.json(
       { success: true, message: `Sent test ${emailId} email` },
-      { status: 200 }
+      { status: 200 },
     );
   }
 }

diff --git a/src/app/(nextjs_migration)/(authenticated)/user/breaches/page.tsx b/src/app/(nextjs_migration)/(authenticated)/user/breaches/page.tsx
@@ -51,7 +51,7 @@ function createEmailOptions({ breachesData, emailSelectIndex }: UserBreaches) {
     (email, index) =>
       `<option ${
         emailSelectIndex === index ? "selected" : ""
-      }>${email}</option>`
+      }>${email}</option>`,
   );
 
   return optionElements.join("");
@@ -126,7 +126,7 @@ export default async function UserBreaches() {
               dangerouslySetInnerHTML={{
                 __html: l10n.getString("breach-heading-email", {
                   "email-select": `<custom-select name="email-account">${createEmailOptions(
-                    userBreachesData
+                    userBreachesData,
                   )}</custom-select>`,
                 }),
               }}

diff --git a/src/app/(nextjs_migration)/(authenticated)/user/settings/page.tsx b/src/app/(nextjs_migration)/(authenticated)/user/settings/page.tsx
@@ -59,7 +59,7 @@ const deleteButton = (email: EmailRow) => {
 
 const createEmailItem = (
   email: EmailRow & { primary?: boolean },
-  breachCounts: Map<string, number>
+  breachCounts: Map<string, number>,
 ) => {
   const l10n = getL10n();
 
@@ -98,12 +98,12 @@ const getSortedEmails = (emails: Array<EmailRow & { primary?: boolean }>) =>
 
 const createEmailList = (
   emails: EmailRow[],
-  breachCounts: Map<string, number>
+  breachCounts: Map<string, number>,
 ) => {
   return (
     <ul className="settings-email-list">
       {getSortedEmails(emails).map((email) =>
-        createEmailItem(email, breachCounts)
+        createEmailItem(email, breachCounts),
       )}
     </ul>
   );
@@ -180,7 +180,7 @@ export default async function Settings() {
     const breaches = await getBreachesForEmail(
       getSha1(email.email),
       allBreaches,
-      true
+      true,
     );
     breachCounts.set(email.email, breaches?.length || 0);
   }
@@ -252,7 +252,7 @@ export default async function Settings() {
                 {l10n.getString(
                   enabledFlags.includes("FxaRebrand")
                     ? "settings-deactivate-account-info-2"
-                    : "settings-deactivate-account-info"
+                    : "settings-deactivate-account-info",
                 )}
               </p>
               <a
@@ -264,7 +264,7 @@ export default async function Settings() {
                 {l10n.getString(
                   enabledFlags.includes("FxaRebrand")
                     ? "settings-fxa-link-label-2"
-                    : "settings-fxa-link-label"
+                    : "settings-fxa-link-label",
                 )}
               </a>
             </section>

diff --git a/src/app/(nextjs_migration)/(guest)/breach-details/[breachName]/page.tsx b/src/app/(nextjs_migration)/(guest)/breach-details/[breachName]/page.tsx
@@ -125,7 +125,7 @@ export default async function BreachDetail(props: {
                   year: "numeric",
                   month: "long",
                   day: "numeric",
-                }
+                },
               ),
               breachTitle: breach.Title,
               addedDate: (breach.AddedDate as unknown as Date).toLocaleString(
@@ -134,7 +134,7 @@ export default async function BreachDetail(props: {
                   year: "numeric",
                   month: "long",
                   day: "numeric",
-                }
+                },
               ),
             })}
           </div>
@@ -251,7 +251,7 @@ function getBreachDetail(categoryId: ReturnType<typeof getBreachCategory>) {
 }
 
 function makeBreachDetail(
-  breachCategory: ReturnType<typeof getBreachCategory>
+  breachCategory: ReturnType<typeof getBreachCategory>,
 ) {
   const breachDetail = getBreachDetail(breachCategory);
   return (
@@ -299,13 +299,13 @@ function getSortedDataClasses(
   isUserBrowserFirefox = false,
   isUserLocaleEnUs = false,
   _isUserLocalEn = false,
-  changePWLink = false
+  changePWLink = false,
 ) {
   const l10n = getL10n();
   const priorityDataClasses: any = getAllPriorityDataClasses(
     isUserBrowserFirefox,
     isUserLocaleEnUs,
-    changePWLink
+    changePWLink,
   );
 
   const sortedDataClasses = {
@@ -327,7 +327,7 @@ function getSortedDataClasses(
     return b.weight - a.weight;
   });
   sortedDataClasses.lowerPriority = sortedDataClasses.lowerPriority.join(
-    ", "
+    ", ",
   ) as any;
   return sortedDataClasses;
 }
@@ -362,30 +362,31 @@ function makeRecommendationCards(breach: HibpLikeDbBreach) {
   const l10n = getL10n();
   const dataClasses = getSortedDataClasses(breach);
 
-  const priorityRecs = dataClasses.priority.map((dataClass) =>
-    // This code predates TypeScript:
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-return
-    dataClass.recommendations?.map((r: any) => (
-      <div
-        key={r.ctaHref}
-        className={`breach-detail-recommendation ${
-          r.recIconClassName as string
-        }`}
-      >
-        <dt>{l10n.getString(r.recommendationCopy.subhead)}</dt>
-        <dd>
-          <p>{l10n.getString(r.recommendationCopy.body)}</p>
-          {r.recommendationCopy.cta ? (
-            <a
-              href={r.ctaHref}
-              target={r.ctaShouldOpenInNewTab ? "_blank" : "_self"}
-            >
-              {l10n.getString(r.recommendationCopy.cta)}
-            </a>
-          ) : null}
-        </dd>
-      </div>
-    ))
+  const priorityRecs = dataClasses.priority.map(
+    (dataClass) =>
+      // This code predates TypeScript:
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-return
+      dataClass.recommendations?.map((r: any) => (
+        <div
+          key={r.ctaHref}
+          className={`breach-detail-recommendation ${
+            r.recIconClassName as string
+          }`}
+        >
+          <dt>{l10n.getString(r.recommendationCopy.subhead)}</dt>
+          <dd>
+            <p>{l10n.getString(r.recommendationCopy.body)}</p>
+            {r.recommendationCopy.cta ? (
+              <a
+                href={r.ctaHref}
+                target={r.ctaShouldOpenInNewTab ? "_blank" : "_self"}
+              >
+                {l10n.getString(r.recommendationCopy.cta)}
+              </a>
+            ) : null}
+          </dd>
+        </div>
+      )),
   );
 
   const genericRecs = getAllGenericRecommendations().map((dataClass: any) => (

diff --git a/src/app/(nextjs_migration)/(guest)/breaches/page.tsx b/src/app/(nextjs_migration)/(guest)/breaches/page.tsx
@@ -83,7 +83,7 @@ export default async function PublicScan() {
                   new Date(a.AddedDate).getTime() <
                   new Date(b.AddedDate).getTime()
                     ? 1
-                    : -1
+                    : -1,
                 )
                 .map((breach) => (
                   <BreachCard
@@ -124,7 +124,7 @@ function BreachCard(props: { breach: HibpLikeDbBreach | Breach }) {
                   year: "numeric",
                   month: "long",
                   day: "numeric",
-                }
+                },
               )}
             </dd>
           </div>
@@ -133,7 +133,7 @@ function BreachCard(props: { breach: HibpLikeDbBreach | Breach }) {
             <dd>
               {formatList(
                 props.breach.DataClasses.map((a: string) => l10n.getString(a)),
-                getLocale(l10n)
+                getLocale(l10n),
               )}
             </dd>
           </div>

diff --git a/src/app/(nextjs_migration)/(guest)/page.tsx b/src/app/(nextjs_migration)/(guest)/page.tsx
@@ -47,7 +47,7 @@ export default function Home() {
               name="email"
               type="email"
               placeholder={l10n.getString(
-                "exposure-landing-hero-email-placeholder"
+                "exposure-landing-hero-email-placeholder",
               )}
               required
             />