From ed129ee56d4647631968242c0655d7ea9c924f3f Mon Sep 17 00:00:00 2001 From: Sascha Date: Sun, 20 Sep 2026 14:42:40 +0200 Subject: [PATCH] refactor: clean up code formatting and improve readability in audit and email service files --- backend/routes/audit.js | 25 +++++++++++++++++++------ backend/services/emailService.js | 4 +++- src/config/locales/en/TextConfig.ts | 8 ++++---- src/scss/sections/quickcheck-page.scss | 1 - 4 files changed, 26 insertions(+), 12 deletions(-) diff --git a/backend/routes/audit.js b/backend/routes/audit.js index ab0c4b5..3896b75 100644 --- a/backend/routes/audit.js +++ b/backend/routes/audit.js @@ -1,7 +1,10 @@ import express from 'express' import rateLimit from 'express-rate-limit' import { body, validationResult } from 'express-validator' -import { sendAuditRequestEmail, sendQuickcheckRequestEmail } from '../services/emailService.js' +import { + sendAuditRequestEmail, + sendQuickcheckRequestEmail, +} from '../services/emailService.js' const router = express.Router() @@ -50,11 +53,14 @@ const validateQuickcheckRequest = [ ] const validateRequest = (req, _res, next) => { - const validators = req.body.formType === 'quickcheck' - ? validateQuickcheckRequest - : validateAuditRequest + const validators = + req.body.formType === 'quickcheck' + ? validateQuickcheckRequest + : validateAuditRequest - Promise.all(validators.map((validator) => validator.run(req))).then(() => next()) + Promise.all(validators.map((validator) => validator.run(req))).then(() => + next(), + ) } router.post('/', auditLimiter, validateRequest, async (req, res) => { @@ -69,7 +75,14 @@ router.post('/', auditLimiter, validateRequest, async (req, res) => { }) } - const { formType = 'audit', name, email, businessConfirmation, gdprConsent, website } = req.body + const { + formType = 'audit', + name, + email, + businessConfirmation, + gdprConsent, + website, + } = req.body if (website) { console.warn('Honeypot triggered, discarding submission.') diff --git a/backend/services/emailService.js b/backend/services/emailService.js index b1c2311..2714358 100644 --- a/backend/services/emailService.js +++ b/backend/services/emailService.js @@ -35,7 +35,9 @@ function getTransporter() { * when it is used in Subject or Reply-To. */ function singleLine(value) { - return String(value).replace(/[\r\n]+/g, ' ').trim() + return String(value) + .replace(/[\r\n]+/g, ' ') + .trim() } /** Minimal HTML escaping - the payload is attacker-controlled by definition. */ diff --git a/src/config/locales/en/TextConfig.ts b/src/config/locales/en/TextConfig.ts index 9f014a5..7ac4a3f 100644 --- a/src/config/locales/en/TextConfig.ts +++ b/src/config/locales/en/TextConfig.ts @@ -365,22 +365,22 @@ export interface TextConfig { problem: { title: string; intro: string; - items: { text: string }[]; + items: { text: string; }[]; outro: string; }; deliverable: { title: string; intro: string; - items: { title: string; text: string }[]; + items: { title: string; text: string; }[]; }; process: { title: string; - steps: { title: string; text: string }[]; + steps: { title: string; text: string; }[]; }; limits: { title: string; intro: string; - items: { text: string }[]; + items: { text: string; }[]; outro: string; }; form: { diff --git a/src/scss/sections/quickcheck-page.scss b/src/scss/sections/quickcheck-page.scss index 8838db8..62a251a 100644 --- a/src/scss/sections/quickcheck-page.scss +++ b/src/scss/sections/quickcheck-page.scss @@ -258,7 +258,6 @@ &__features { grid-template-columns: repeat(3, 1fr); } - } @media (prefers-reduced-motion: reduce) {