feat: add form for submitting testimonials (#3)
All checks were successful
Node.js CI / Lint and Test (push) Successful in 1m48s

### Explanation

_No response_

### Issue

_No response_

### Attestations

- [x] I have read and agree to the [Code of Conduct](https://docs.nhcarrigan.com/community/coc/)
- [x] I have read and agree to the [Community Guidelines](https://docs.nhcarrigan.com/community/guide/).
- [x] My contribution complies with the [Contributor Covenant](https://docs.nhcarrigan.com/dev/covenant/).

### Dependencies

- [ ] I have pinned the dependencies to a specific patch version.

### Style

- [x] I have run the linter and resolved any errors.
- [x] My pull request uses an appropriate title, matching the conventional commit standards.
- [x] My scope of feat/fix/chore/etc. correctly matches the nature of changes in my pull request.

### Tests

- [ ] My contribution adds new code, and I have added tests to cover it.
- [ ] My contribution modifies existing code, and I have updated the tests to reflect these changes.
- [ ] All new and existing tests pass locally with my changes.
- [ ] Code coverage remains at or above the configured threshold.

### Documentation

_No response_

### Versioning

Minor - My pull request introduces a new non-breaking feature.

Reviewed-on: #3
Co-authored-by: Naomi Carrigan <commits@nhcarrigan.com>
Co-committed-by: Naomi Carrigan <commits@nhcarrigan.com>
This commit is contained in:
Naomi Carrigan 2025-02-20 16:07:39 -08:00 committed by Naomi Carrigan
parent 439df9968a
commit f8662980af
16 changed files with 258 additions and 9 deletions

View File

@ -16,6 +16,7 @@ import type {
SuccessResponse,
Staff,
DataResponse,
Testimonial,
} from "@repo/types";
@Injectable({
@ -142,6 +143,20 @@ export class ApiService {
return response;
}
public async submitTestimonial(
testimonial: Partial<Testimonial>,
): Promise<SuccessResponse | ErrorResponse> {
const request = await fetch(`${this.url}/submit/testimonials`, {
body: JSON.stringify(testimonial),
headers: {
"Content-type": "application/json",
},
method: "POST",
});
const response = await request.json() as SuccessResponse | ErrorResponse;
return response;
}
public async getData(
type:
| "appeals"
@ -150,7 +165,8 @@ export class ApiService {
| "events"
| "meetings"
| "mentorships"
| "staff",
| "staff"
| "testimonials",
token: string,
): Promise<DataResponse | ErrorResponse> {
const request = await fetch(`${this.url}/list/${type}`, {
@ -172,7 +188,8 @@ export class ApiService {
| "events"
| "meetings"
| "mentorships"
| "staff",
| "staff"
| "testimonials",
id: string,
token: string,
): Promise<SuccessResponse | ErrorResponse> {

View File

@ -13,6 +13,8 @@ import { MeetingComponent } from "./forms/meeting/meeting.component.js";
import { MentorshipComponent }
from "./forms/mentorship/mentorship.component.js";
import { StaffComponent } from "./forms/staff/staff.component.js";
import { TestimonialComponent }
from "./forms/testimonial/testimonial.component.js";
import { HomeComponent } from "./home/home.component.js";
import { ReviewComponent } from "./review/review.component.js";
import type { Routes } from "@angular/router";
@ -27,4 +29,5 @@ export const routes: Routes = [
{ component: MentorshipComponent, path: "mentorship" },
{ component: StaffComponent, path: "staff" },
{ component: ReviewComponent, path: "review" },
{ component: TestimonialComponent, path: "testimonial" },
];

View File

@ -0,0 +1,12 @@
<h1>Testimonials</h1>
<p>This form allows past clients and colleagues to submit reviews reflecting their positive experience with our work.</p>
<p>We will respond to these submissions when your testimonial is <a href="https://testimonials.nhcarrigan.com" target="_blank">on our site</a>.</p>
<app-error *ngIf="error" error="{{ error }}"></app-error>
<app-success *ngIf="success"></app-success>
<form *ngIf="!loading">
<app-userinfo [firstNameControl]="firstName" [lastNameControl]="lastName" [emailControl]="email" [companyControl]="company"></app-userinfo>
<app-multi-line label="Share your experience working with us!" [control]="content"></app-multi-line>
<app-consent [control]="consent"></app-consent>
<button type="button" (click)="submit($event)">Submit</button>
</form>
<a routerLink="/">Back to home</a>

View File

@ -0,0 +1,80 @@
/**
* @copyright nhcarrigan
* @license Naomi's Public License
* @author Naomi Carrigan
*/
import { CommonModule } from "@angular/common";
import { Component } from "@angular/core";
import { FormControl, ReactiveFormsModule } from "@angular/forms";
import { RouterModule } from "@angular/router";
import { ApiService } from "../../api.service.js";
import { ConsentComponent } from "../../consent/consent.component.js";
import { ErrorComponent } from "../../error/error.component.js";
import { MultiLineComponent }
from "../../inputs/multi-line/multi-line.component.js";
import { SuccessComponent } from "../../success/success.component.js";
import { UserinfoComponent } from "../../userinfo/userinfo.component.js";
@Component({
imports: [
RouterModule,
CommonModule,
ConsentComponent,
UserinfoComponent,
ErrorComponent,
ReactiveFormsModule,
MultiLineComponent,
SuccessComponent,
],
selector: "app-testimonial",
styleUrl: "./testimonial.component.css",
templateUrl: "./testimonial.component.html",
})
export class TestimonialComponent {
public loading = false;
public error = "";
public success = false;
public firstName = new FormControl("");
public lastName = new FormControl("");
public email = new FormControl("");
public company = new FormControl("");
public content = new FormControl("");
public consent = new FormControl(false);
public constructor(private readonly apiService: ApiService) {}
public submit(event: MouseEvent): void {
this.error = "";
const { form } = event.target as HTMLButtonElement;
const valid = form?.reportValidity();
if (valid !== true) {
return;
}
this.loading = true;
this.apiService.
submitTestimonial({
consent: this.consent.value ?? false,
content: this.content.value ?? undefined,
email: this.email.value ?? undefined,
firstName: this.firstName.value ?? undefined,
lastName: this.lastName.value ?? undefined,
}).
then((response) => {
if ("error" in response) {
this.error = response.error;
this.loading = false;
} else {
this.error = "";
this.success = true;
}
}).
catch(() => {
this.error = "An error occurred while submitting the form.";
this.loading = false;
});
}
}

View File

@ -29,3 +29,6 @@
<a routerLink="/mentorship">
<i class="fa-solid fa-brain" aria-hidden="true"></i> Mentorship Programme
</a>
<a routerLink="/testimonial">
<i class="fa-solid fa-star-half-stroke" aria-hidden="true"></i> Testimonials
</a>

View File

@ -58,11 +58,21 @@
>
Staff Applications
</button>
<button
[disabled]="view === 'testimonials'"
type="button"
(click)="setView('testimonials')"
>
Testimonials
</button>
<h2>{{ view }}</h2>
<div *ngFor="let datum of data">
<h3>{{ datum.email }}</h3>
<div *ngFor="let obj of datum.info">
<p><strong>{{ obj.key }}</strong>: {{ obj.value }}</p>
<p>
<strong>{{ obj.key }}</strong
>: {{ obj.value }}
</p>
</div>
<button type="button" (click)="markReviewed(datum.id)">
Mark as Reviewed

View File

@ -41,7 +41,8 @@ export class ReviewComponent {
| "events"
| "meetings"
| "mentorships"
| "staff" = "";
| "staff"
| "testimonials" = "";
public constructor(private readonly apiService: ApiService) {
const storedToken = localStorage.getItem("token");
@ -98,7 +99,8 @@ export class ReviewComponent {
| "events"
| "meetings"
| "mentorships"
| "staff";
| "staff"
| "testimonials";
void this.apiService.
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- We know token is not null
markReviewed(view, id, this.token.value!).then((data) => {
@ -120,7 +122,8 @@ export class ReviewComponent {
| "events"
| "meetings"
| "mentorships"
| "staff",
| "staff"
| "testimonials",
): void {
this.view = view;
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- We know token is not null

View File

@ -0,0 +1,13 @@
/**
* @copyright nhcarrigan
* @license Naomi's Public License
* @author Naomi Carrigan
*/
export interface Testimonial {
consent: boolean;
content: string;
email: string;
firstName: string;
lastName: string;
}

View File

@ -11,6 +11,7 @@ import type { Event } from "./forms/event.js";
import type { Meeting } from "./forms/meeting.js";
import type { Mentorship } from "./forms/mentorship.js";
import type { Staff } from "./forms/staff.js";
import type { Testimonial } from "./forms/testimonial.js";
import type { ReviewRequest } from "./requests/review.js";
import type { DataResponse } from "./responses/data.js";
import type { ErrorResponse } from "./responses/error.js";
@ -28,4 +29,5 @@ export type {
DataResponse,
ErrorResponse,
SuccessResponse,
Testimonial,
};

View File

@ -103,3 +103,12 @@ model Mentorships {
paymentUnderstanding Boolean
createdAt DateTime @default(now())
}
model Testimonials {
id String @id @default(auto()) @map("_id") @db.ObjectId
email String @unique
firstName String
lastName String
content String
createdAt DateTime @default(now())
}

View File

@ -0,0 +1,64 @@
/**
* @copyright nhcarrigan
* @license Naomi's Public License
* @author Naomi Carrigan
*/
import { validateBody } from "../../modules/validateBody.js";
import { logger } from "../../utils/logger.js";
import { sendMail } from "../../utils/mailer.js";
import type { PrismaClient } from "@prisma/client";
import type { Testimonial, ErrorResponse, SuccessResponse } from "@repo/types";
import type { FastifyReply, FastifyRequest } from "fastify";
/**
*Handles testimonial form submissions.
* @param database - The Prisma database client.
* @param request - The request object.
* @param response - The Fastify reply utility.
*/
export const submitTestimonialHandler = async(
database: PrismaClient,
request: FastifyRequest<{ Body: Testimonial }>,
response: FastifyReply<{ Reply: SuccessResponse | ErrorResponse }>,
): Promise<void> => {
try {
const isInvalid = validateBody(
// eslint-disable-next-line @typescript-eslint/consistent-type-assertions -- We're passing a narrower type and TS hates that?
request.body as unknown as Record<string, unknown>,
"testimonials",
);
if (isInvalid !== null) {
await response.status(400).send({ error: isInvalid });
return;
}
const exists = await database.staff.findUnique({
where: {
email: request.body.email,
},
});
if (exists !== null) {
await response.
status(429).
send({
error:
// eslint-disable-next-line stylistic/max-len -- This is a long string.
"You have already submitted a staff application. Please wait for it to be reviewed.",
});
return;
}
const data = { ...request.body };
// @ts-expect-error -- We're deleting a property here.
delete data.consent;
await database.testimonials.create({
data,
});
await sendMail("testimonials", data);
await response.send({ success: true });
} catch (error) {
// eslint-disable-next-line @typescript-eslint/consistent-type-assertions -- we bein' lazy.
await logger.error("/submit/testimonials", error as Error);
await response.status(500).send({ error: error instanceof Error
? error.message
: "Internal Server Error" });
}
};

View File

@ -9,4 +9,5 @@ export type DatabasePath = | "appeals"
| "events"
| "meetings"
| "mentorships"
| "staff";
| "staff"
| "testimonials";

View File

@ -14,6 +14,7 @@ import type {
Mentorships,
PrismaClient,
Staff,
Testimonials,
} from "@prisma/client";
/**
@ -27,10 +28,17 @@ const listUnreviewedSubmissions = async(
route: DatabasePath,
): Promise<
Array<
Appeals | Commissions | Contacts | Events | Meetings | Mentorships | Staff
| Appeals
| Commissions
| Contacts
| Events
| Meetings
| Mentorships
| Staff
| Testimonials
>
> => {
const query = { };
const query = {};
switch (route) {
case "appeals":
return await database.appeals.findMany(query);
@ -46,6 +54,8 @@ const listUnreviewedSubmissions = async(
return await database.mentorships.findMany(query);
case "staff":
return await database.staff.findMany(query);
case "testimonials":
return await database.testimonials.findMany(query);
default:
return [];
}
@ -79,6 +89,8 @@ const checkSubmissionExists = async(
return Boolean(await database.mentorships.findUnique(query));
case "staff":
return Boolean(await database.staff.findUnique(query));
case "testimonials":
return Boolean(await database.testimonials.findUnique(query));
default:
return false;
}
@ -122,6 +134,9 @@ const markSubmissionReviewed = async(
case "staff":
await database.staff.delete(update);
break;
case "testimonials":
await database.testimonials.delete(update);
break;
default:
break;
}

View File

@ -70,6 +70,12 @@ const validators: Record<DatabasePath, Array<string>> = {
"difficultSituation",
"leadershipSituation",
],
testimonials: [
"firstName",
"lastName",
"email",
"content",
],
};
/**

View File

@ -13,6 +13,8 @@ import { submitMeetingHandler } from "../handlers/submit/meetingHandler.js";
import { submitMentorshipHandler }
from "../handlers/submit/mentorshipHandler.js";
import { submitStaffHandler } from "../handlers/submit/staffHandler.js";
import { submitTestimonialHandler }
from "../handlers/submit/testimonialHandler.js";
import type { WrappedRoute } from "../interfaces/wrappedRoute.js";
import type {
Appeal,
@ -24,6 +26,7 @@ import type {
Staff,
ErrorResponse,
SuccessResponse,
Testimonial,
} from "@repo/types";
/**
@ -31,6 +34,7 @@ import type {
* @param database - The Prisma client.
* @returns A Fastify plugin.
*/
// eslint-disable-next-line max-lines-per-function -- Prisma typings don't allow us to mount these dynamically...
export const submitRoutes: WrappedRoute = (database) => {
return async(fastify) => {
fastify.post<{ Body: Appeal; Reply: SuccessResponse | ErrorResponse }>(
@ -81,5 +85,12 @@ export const submitRoutes: WrappedRoute = (database) => {
await submitStaffHandler(database, request, response);
},
);
fastify.post<{ Body: Testimonial; Reply: SuccessResponse | ErrorResponse }>(
"/submit/testimonials",
async(request, response) => {
await submitTestimonialHandler(database, request, response);
},
);
};
};