Skip to content

Commit

Permalink
Reduce size of email body even more
Browse files Browse the repository at this point in the history
- see previous commit
- Reduce size of boilerplate text even more
- Limit size of user input to a very limited size
  • Loading branch information
mpanne committed Jan 16, 2025
1 parent d65416f commit 195286f
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 13 deletions.
11 changes: 4 additions & 7 deletions packages/dito/app/resources/content.ts
Original file line number Diff line number Diff line change
Expand Up @@ -576,14 +576,16 @@ Beispiele für Akteur:innen sind:
emailLabel: "Ihre E-Mail Adresse (optional)",
policyTitleLabel: "Vorläufiger Arbeitstitel des Vorhabens",
policyTitleRequired: "Bitte geben Sie einen Titel für Ihr Vorhaben an.",
policyTitleTooLong: "Bitte wählen Sie einen kürzeren Titel.",
policyTitleTooLong:
"Bitte wählen Sie einen kürzeren Titel und ergänzen Sie diesen gegebenenfalls direkt in der resultierenden E-Mail.",
precheckAnswersRequired: "Bitte geben Sie die Vorprüfung-Antworten an",
reasonLabel: "Begründung",
reasonRequired:
"Bitte geben Sie eine Begründung für den fehlenden Digitalbezug an.",
reasonLong:
"Achtung, Ihre Begründung ist sehr lang. Möglicherweise sehen Sie in dem PDF nicht den gesamten Text. Sie können ihn aber vollständig kopieren, es gehen keine Inhalte verloren.",
reasonTooLong: "Bitte geben Sie eine kürzere Begründung ein.",
reasonTooLong:
"Bitte geben Sie eine kürzere Begründung ein und ergänzen Sie diese gegebenenfalls direkt in der resultierenden E-Mail.",
faqs: {
title: "Häufige Fragen zum Senden der Vorprüfung",
details: [
Expand Down Expand Up @@ -626,11 +628,6 @@ hiermit erhalten Sie das Ergebnis Ihrer Vorprüfung:`,
bodyAfter: `Mit freundlichen Grüßen
*Diese E-Mail wurde automatisch erstellt.*
------------------------------------------
Der Digitalcheck ist die Prozessbegleitung für digitaltaugliche Gesetze und ab Januar 2025 interoperable Regelungsvorhaben. Dieser Auftrag wurde dem Digitalcheck vom BMI erteilt.
Bei inhaltlichen Anliegen zu Ihrem Regelungsvorhaben helfen wir Ihnen gerne weiter. Schreiben Sie uns über [email protected] oder rufen Sie uns an unter 0151/40 76 78 39.
`,
},
downloadStarted: "Vorprüfung wird heruntergeladen",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,14 +7,14 @@ const positiveValidation = z.object({
title: z
.string()
.min(1, { message: preCheck.result.form.policyTitleRequired })
.max(500, { message: preCheck.result.form.policyTitleTooLong }),
.max(100, { message: preCheck.result.form.policyTitleTooLong }),
});

const negativeValidation = positiveValidation.extend({
negativeReasoning: z
.string()
.min(1, { message: preCheck.result.form.reasonRequired })
.max(5000, { message: preCheck.result.form.reasonTooLong }),
.max(500, { message: preCheck.result.form.reasonTooLong }),
});

export default function getResultValidatorForAnswers(answers: PreCheckAnswers) {
Expand Down
2 changes: 1 addition & 1 deletion packages/dito/app/routes/vorpruefung.ergebnis/route.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ function buildEmailBody(
? preCheck.result.form.reasonLabel + ":\n\n" + negativeReasoning + "\n\n"
: "";

return `${emailTemplate.bodyBefore}\n${resultText}${emailTemplate.bodyAfter}`;
return `${emailTemplate.bodyBefore}\n${resultText}\n\n${emailTemplate.bodyAfter}`;
}

function resolveRecipients(result: PreCheckResult) {
Expand Down
6 changes: 3 additions & 3 deletions packages/dito/tests/e2e/assessment.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ test.describe("test positive assessment page", () => {
});

test("title can't be too long", async ({ page }) => {
await page.getByLabel("Arbeitstitel des Vorhabens").fill("A".repeat(501));
await page.getByLabel("Arbeitstitel des Vorhabens").fill("A".repeat(101));
await interceptMailToRedirectAndExpect(page);
await page.getByTestId("result-email-button").click();
await expect(page.getByTestId("title-error")).toBeVisible();
Expand Down Expand Up @@ -127,8 +127,8 @@ test.describe("test form in negative case", () => {
});

test("title and reasoning can't be too long", async ({ page }) => {
await page.getByLabel("Begründung").fill("A".repeat(5001));
await page.getByLabel("Arbeitstitel des Vorhabens").fill("B".repeat(501));
await page.getByLabel("Begründung").fill("A".repeat(501));
await page.getByLabel("Arbeitstitel des Vorhabens").fill("B".repeat(101));
await interceptMailToRedirectAndExpect(page);
await page.getByTestId("result-email-button").click();
await expect(page.getByTestId("title-error")).toBeVisible();
Expand Down

0 comments on commit 195286f

Please sign in to comment.