feat: add an ability to mark contributors as hidden gf-346

apps/backend/src/db/migrations/20240924094947_add_hidden_at_to_contributors_table.ts

@@ -0,0 +1,21 @@
+import { type Knex } from "knex";
+
+const TABLE_NAME = "contributors";
+
+const ColumnName = {
+	HIDDEN_AT: "hidden_at",
+} as const;
+
+function up(knex: Knex): Promise<void> {
+	return knex.schema.alterTable(TABLE_NAME, (table) => {
+		table.dateTime(ColumnName.HIDDEN_AT).nullable().defaultTo(null);
+	});
+}
+
+function down(knex: Knex): Promise<void> {
+	return knex.schema.alterTable(TABLE_NAME, (table) => {
+		table.dropColumn(ColumnName.HIDDEN_AT);
+	});
+}
+
+export { down, up };

apps/backend/src/modules/activity-logs/activity-log.repository.ts

@@ -47,10 +47,12 @@ class ActivityLogRepository implements Repository {
 	}: ActivityLogQueryParameters): Promise<{ items: ActivityLogEntity[] }> {
 		const query = this.activityLogModel
 			.query()
-			.withGraphFetched("[gitEmail.contributor, project, createdByUser]")
+			.withGraphFetched("[project, createdByUser]")
+			.withGraphJoined("gitEmail.contributor")
 			.modifyGraph("gitEmail.contributor", (builder) => {
-				builder.select("id", "name");
+				builder.select("id", "name", "hiddenAt");
 			})
+			.whereNull("gitEmail:contributor.hiddenAt")
 			.whereBetween("activity_logs.date", [startDate, endDate])
 			.orderBy("date");
 

apps/backend/src/modules/activity-logs/activity-log.service.ts

@@ -147,8 +147,11 @@ class ActivityLogService implements Service {
 		);
 
 		const allContributors = await (projectId
-			? this.contributorService.findAllByProjectId(Number(projectId))
-			: this.contributorService.findAllWithoutPagination());
+			? this.contributorService.findAllByProjectId({
+					hasHidden: false,
+					projectId: Number(projectId),
+				})
+			: this.contributorService.findAllWithoutPagination({ hasHidden: false }));
 
 		const dateRange = getDateRange(startDate, endDate);
 
@@ -184,8 +187,11 @@ class ActivityLogService implements Service {
 
 				return {
 					commitsNumber: commitsArray,
-					contributorId: contributorId ?? "",
-					contributorName: contributorName ?? "",
+					contributor: {
+						hiddenAt: null,
+						id: contributorId ?? "",
+						name: contributorName ?? "",
+					},
 				};
 			}),
 		};

apps/backend/src/modules/contributors/contributor.controller.ts

@@ -174,7 +174,10 @@ class ContributorController extends BaseController {
 			const projectId = Number(options.query.projectId);
 
 			return {
-				payload: await this.contributorService.findAllByProjectId(projectId),
+				payload: await this.contributorService.findAllByProjectId({
+					hasHidden: true,
+					projectId,
+				}),
 				status: HTTPCode.OK,
 			};
 		}

apps/backend/src/modules/contributors/contributor.entity.ts

@@ -4,46 +4,53 @@ import { type ContributorGetAllItemResponseDto } from "./libs/types/types.js";
 
 class ContributorEntity implements Entity {
 	private gitEmails: { email: string; id: number }[];
+	private hiddenAt: null | string;
 	private id: null | number;
 	private lastActivityDate: null | string;
 	private name: string;
 	private projects: { id: number; name: string }[];
 
 	private constructor({
 		gitEmails,
+		hiddenAt,
 		id,
 		lastActivityDate,
 		name,
 		projects,
 	}: {
 		gitEmails: { email: string; id: number }[];
+		hiddenAt: null | string;
 		id: null | number;
 		lastActivityDate: null | string;
 		name: string;
 		projects: { id: number; name: string }[];
 	}) {
+		this.gitEmails = gitEmails;
 		this.id = id;
-		this.name = name;
+		this.hiddenAt = hiddenAt;
 		this.lastActivityDate = lastActivityDate;
-		this.gitEmails = gitEmails;
+		this.name = name;
 		this.projects = projects;
 	}
 
 	public static initialize({
 		gitEmails,
+		hiddenAt,
 		id,
 		lastActivityDate,
 		name,
 		projects,
 	}: {
 		gitEmails: { email: string; id: number }[];
+		hiddenAt: null | string;
 		id: number;
 		lastActivityDate: null | string;
 		name: string;
 		projects: { id: number; name: string }[];
 	}): ContributorEntity {
 		return new ContributorEntity({
 			gitEmails,
+			hiddenAt,
 			id,
 			lastActivityDate,
 			name,
@@ -53,15 +60,18 @@ class ContributorEntity implements Entity {
 
 	public static initializeNew({
 		gitEmails = [],
+		hiddenAt = null,
 		name,
 		projects = [],
 	}: {
 		gitEmails?: { email: string; id: number }[];
+		hiddenAt?: null | string;
 		name: string;
 		projects?: { id: number; name: string }[];
 	}): ContributorEntity {
 		return new ContributorEntity({
 			gitEmails,
+			hiddenAt,
 			id: null,
 			lastActivityDate: null,
 			name,
@@ -71,12 +81,14 @@ class ContributorEntity implements Entity {
 
 	public toNewObject(): {
 		gitEmails: { email: string; id: number }[];
+		hiddenAt: null | string;
 		lastActivityDate: null | string;
 		name: string;
 		projects: { id: number; name: string }[];
 	} {
 		return {
 			gitEmails: this.gitEmails,
+			hiddenAt: this.hiddenAt,
 			lastActivityDate: this.lastActivityDate,
 			name: this.name,
 			projects: this.projects,
@@ -86,6 +98,7 @@ class ContributorEntity implements Entity {
 	public toObject(): ContributorGetAllItemResponseDto {
 		return {
 			gitEmails: this.gitEmails,
+			hiddenAt: this.hiddenAt,
 			id: this.id as number,
 			lastActivityDate: this.lastActivityDate,
 			name: this.name,

apps/backend/src/modules/contributors/contributor.model.ts

@@ -10,8 +10,13 @@ import { type ProjectModel } from "../projects/project.model.js";
 
 class ContributorModel extends AbstractModel {
 	public gitEmails!: GitEmailModel[];
+
+	public hiddenAt!: null | string;
+
 	public lastActivityDate!: null | string;
+
 	public name!: string;
+
 	public projects!: ProjectModel[];
 
 	public static override get relationMappings(): RelationMappings {

apps/backend/src/modules/contributors/contributor.repository.ts

@@ -56,12 +56,13 @@ class ContributorRepository implements Repository {
 	}
 
 	public async findAll({
+		hasHidden = true,
 		page,
 		pageSize,
-	}: PaginationQueryParameters): Promise<
+	}: { hasHidden?: boolean } & PaginationQueryParameters): Promise<
 		PaginationResponseDto<ContributorEntity>
 	> {
-		const { results, total } = await this.contributorModel
+		const query = this.contributorModel
 			.query()
 			.orderBy("createdAt", SortType.DESCENDING)
 			.page(page, pageSize)
@@ -77,6 +78,12 @@ class ContributorRepository implements Repository {
 			.groupBy("contributors.id")
 			.withGraphFetched("gitEmails");
 
+		if (!hasHidden) {
+			query.whereNull("contributors.hiddenAt");
+		}
+
+		const { results, total } = await query;
+
 		return {
 			items: results.map((contributor) => {
 				return ContributorEntity.initialize(contributor);
@@ -85,10 +92,14 @@ class ContributorRepository implements Repository {
 		};
 	}
 
-	public async findAllByProjectId(
-		projectId: number,
-	): Promise<{ items: ContributorEntity[] }> {
-		const contributorsWithProjectsAndEmails = await this.contributorModel
+	public async findAllByProjectId({
+		hasHidden = true,
+		projectId,
+	}: {
+		hasHidden?: boolean;
+		projectId: number;
+	}): Promise<{ items: ContributorEntity[] }> {
+		const query = this.contributorModel
 			.query()
 			.select("contributors.*")
 			.select(
@@ -105,20 +116,31 @@ class ContributorRepository implements Repository {
 			.leftJoin("activity_logs", "git_emails.id", "activity_logs.git_email_id")
 			.leftJoin("projects", "activity_logs.project_id", "projects.id")
 			.where("projects.id", projectId)
+			.whereNull("contributors.hiddenAt")
 			.groupBy("contributors.id")
 			.withGraphFetched("gitEmails");
 
+		if (!hasHidden) {
+			query.whereNull("contributors.hiddenAt");
+		}
+
+		const contributorsWithProjectsAndEmails = await query;
+
 		return {
 			items: contributorsWithProjectsAndEmails.map((contributor) => {
 				return ContributorEntity.initialize(contributor);
 			}),
 		};
 	}
 
-	public async findAllWithoutPagination(): Promise<{
+	public async findAllWithoutPagination({
+		hasHidden = true,
+	}: {
+		hasHidden?: boolean;
+	}): Promise<{
 		items: ContributorEntity[];
 	}> {
-		const results = await this.contributorModel
+		const query = this.contributorModel
 			.query()
 			.select("contributors.*")
 			.select(
@@ -132,6 +154,12 @@ class ContributorRepository implements Repository {
 			.groupBy("contributors.id")
 			.withGraphFetched("gitEmails");
 
+		if (!hasHidden) {
+			query.whereNull("contributors.hiddenAt");
+		}
+
+		const results = await query;
+
 		return {
 			items: results.map((contributor) => {
 				return ContributorEntity.initialize(contributor);
@@ -208,9 +236,13 @@ class ContributorRepository implements Repository {
 		contributorId: number,
 		data: ContributorPatchRequestDto,
 	): Promise<ContributorEntity | null> {
+		const hiddenAt = data.isHidden ? new Date().toISOString() : null;
 		const contributor = await this.contributorModel
 			.query()
-			.patchAndFetchById(contributorId, { name: data.name });
+			.patchAndFetchById(contributorId, {
+				hiddenAt,
+				name: data.name,
+			});
 
 		return ContributorEntity.initialize(contributor);
 	}

apps/backend/src/modules/contributors/contributor.service.ts

@@ -61,9 +61,10 @@ class ContributorService implements Service {
 	}
 
 	public async findAll(
-		parameters: PaginationQueryParameters,
+		parameters: { hasHidden?: boolean } & PaginationQueryParameters,
 	): Promise<ContributorGetAllResponseDto> {
 		const contributors = await this.contributorRepository.findAll({
+			hasHidden: parameters.hasHidden ?? true,
 			page: parameters.page - PAGE_INDEX_OFFSET,
 			pageSize: parameters.pageSize,
 		});
@@ -84,11 +85,17 @@ class ContributorService implements Service {
 		};
 	}
 
-	public async findAllByProjectId(
-		projectId: number,
-	): Promise<ContributorGetAllResponseDto> {
-		const contributors =
-			await this.contributorRepository.findAllByProjectId(projectId);
+	public async findAllByProjectId({
+		hasHidden = true,
+		projectId,
+	}: {
+		hasHidden?: boolean;
+		projectId: number;
+	}): Promise<ContributorGetAllResponseDto> {
+		const contributors = await this.contributorRepository.findAllByProjectId({
+			hasHidden,
+			projectId,
+		});
 
 		return {
 			items: contributors.items.map((item) => {
@@ -106,9 +113,15 @@ class ContributorService implements Service {
 		};
 	}
 
-	public async findAllWithoutPagination(): Promise<ContributorGetAllResponseDto> {
+	public async findAllWithoutPagination({
+		hasHidden,
+	}: {
+		hasHidden?: boolean;
+	}): Promise<ContributorGetAllResponseDto> {
 		const contributors =
-			await this.contributorRepository.findAllWithoutPagination();
+			await this.contributorRepository.findAllWithoutPagination({
+				hasHidden: hasHidden ?? true,
+			});
 
 		return {
 			items: contributors.items.map((item) => {

apps/frontend/src/libs/components/checkbox/checkbox.tsx

@@ -12,12 +12,14 @@ import styles from "./styles.module.css";
 type Properties<T extends FieldValues> = {
 	control: Control<T, null>;
 	errors: FieldErrors<T>;
+	id?: string;
 	name: FieldPath<T>;
 };
 
 const Checkbox = <T extends FieldValues>({
 	control,
 	errors,
+	id,
 	name,
 }: Properties<T>): JSX.Element => {
 	const { field } = useFormController({ control, name });
@@ -30,6 +32,7 @@ const Checkbox = <T extends FieldValues>({
 			<input
 				className={styles["checkbox"]}
 				defaultChecked={field.value}
+				id={id}
 				name={name}
 				onChange={field.onChange}
 				type="checkbox"

apps/frontend/src/libs/components/icon/libs/maps/icon-name-to-svg.map.ts

@@ -3,6 +3,7 @@ import { type FC } from "react";
 import Access from "~/assets/images/icons/access.svg?react";
 import Analytics from "~/assets/images/icons/analytics.svg?react";
 import Calendar from "~/assets/images/icons/calendar.svg?react";
+import Check from "~/assets/images/icons/check.svg?react";
 import Clipboard from "~/assets/images/icons/clipboard.svg?react";
 import Contributors from "~/assets/images/icons/contributors.svg?react";
 import Cross from "~/assets/images/icons/cross.svg?react";
@@ -27,6 +28,7 @@ const iconNameToSvg: Record<IconName, FC<React.SVGProps<SVGSVGElement>>> = {
 	access: Access,
 	analytics: Analytics,
 	calendar: Calendar,
+	check: Check,
 	clipboard: Clipboard,
 	contributors: Contributors,
 	cross: Cross,