feat: Add share reason

Author: oSumAtrIX

migrations/0002_vote-description.sql

@@ -0,0 +1 @@
+ALTER TABLE votes ADD COLUMN reason TEXT NOT NULL;

src/lib/database.ts

@@ -72,9 +72,9 @@ export class DatabaseService {
 			const voteStatements = vote.participants.map((p) =>
 				this.db
 					.prepare(
-						'INSERT INTO votes (session_id, voter_id, recipient_id, share) VALUES (?, ?, ?, ?)'
+						'INSERT INTO votes (session_id, voter_id, recipient_id, share, reason) VALUES (?, ?, ?, ?, ?)'
 					)
-					.bind(sessionId, vote.voterId, p.id, p.share)
+					.bind(sessionId, vote.voterId, p.id, p.share, p.reason)
 			);
 
 			await this.db.batch(voteStatements);
@@ -111,14 +111,15 @@ export class DatabaseService {
 
 	async getParticipantShares(
 		sessionId: number
-	): Promise<Record<string, Array<number>>> {
+	): Promise<Record<string, Array<{ share: number; reason: string }>>> {
 		const { results } = await this.db
 			.prepare(
 				`
 				SELECT 
 					p.name,
 					p.description,
-					v.share
+					v.share,
+					v.reason
 				FROM participants p
 				LEFT JOIN votes v 
 					ON p.id = v.recipient_id AND v.session_id = ?
@@ -129,16 +130,20 @@ export class DatabaseService {
 			.all<{
 				name: string;
 				share: number;
+				reason: string;
 			}>();
 
-		const participantShares: Record<string, Array<number>> = {};
+		const participantShares: Record<
+			string,
+			Array<{ share: number; reason: string }>
+		> = {};
 
 		results.map((result) => {
-			const { name, share } = result;
+			const { name, share, reason } = result;
 
 			if (!participantShares[name]) participantShares[name] = [];
 
-			if (share != null) participantShares[name].push(share);
+			if (share != null) participantShares[name].push({ share, reason });
 		});
 
 		return participantShares;

src/lib/types.ts

@@ -27,6 +27,7 @@ export interface SubmitVoteRequest {
 	participants: Array<{
 		id: number;
 		share: number;
+		reason: string;
 	}>;
 }
 

src/routes/+page.svelte

@@ -23,6 +23,8 @@
 			name: string;
 			description: string;
 			share: number;
+			reason: string;
+			reasons: string[];
 		}[];
 	} | null = null;
 
@@ -94,9 +96,10 @@
 				},
 				body: JSON.stringify({
 					voterId,
-					participants: session.participants.map(({ id, share }) => ({
+					participants: session.participants.map(({ id, share, reason }) => ({
 						id,
-						share: share || 0
+						share: share || 0,
+						reason
 					}))
 				})
 			});
@@ -331,8 +334,17 @@
 									placeholder="Share"
 									class="input"
 								/>
+								<textarea bind:value={participant.reason} placeholder="Reason"
+								></textarea>
 							{:else}
 								<p><strong>Share:</strong> {participant.share || 0}</p>
+								<strong>Reasons:</strong>
+
+								<ul>
+									{#each participant.reasons as reason}
+										<li>{reason}</li>
+									{/each}
+								</ul>
 							{/if}
 						{:else}
 							<button on:click={() => selectParticipant(participant)}
@@ -377,7 +389,7 @@
 			{/if}
 		</div>
 		{#if session}
-			<button on:click={() => (session = null)}> Go back</button>
+			<button on:click={() => (session = voterId = null)}> Go back</button>
 		{/if}
 	{/if}
 </main>

src/routes/api/[sessionKey]/+server.ts

@@ -28,7 +28,7 @@ export const GET: RequestHandler = async ({ params, request, platform }) => {
 
 		const participantShares: Record<
 			string,
-			Array<number>
+			Array<{ share: number; reason: string }>
 		> = await db.getParticipantShares(sessionId);
 
 		const participantNames = Object.keys(participantShares);
@@ -50,9 +50,10 @@ export const GET: RequestHandler = async ({ params, request, platform }) => {
 		}
 
 		const participants = Object.entries(participantShares).map(
-			([name, shares]) => ({
+			([name, vote]) => ({
 				name,
-				share: shares.reduce((sum, value) => sum + value, 0) / participantLength
+				share: vote.reduce((sum, v) => sum + v.share, 0) / participantLength,
+				reasons: vote.map((v) => v.reason)
 			})
 		);
 
@@ -92,12 +93,30 @@ export const POST: RequestHandler = async ({ params, request, platform }) => {
 			notFoundError();
 		}
 
+		const allReasonsExist = vote.participants.every(
+			(p) => p.reason && p.reason.trim().length > 0
+		);
+		if (!allReasonsExist) {
+			badRequestError(
+				'One or more participants in the vote do not have a share reason'
+			);
+		}
+
 		const participants = await db.getParticipants(session.id);
 
 		if (vote.participants.length !== participants.length - 1) {
 			badRequestError('Invalid number of share participants.');
 		}
 
+		const voterExists = participants.some((p) => p.id === voterId);
+		if (!voterExists) {
+			badRequestError('Voter is not a participant in this session');
+		}
+
+		if (vote.participants.some((s) => s.id === voterId)) {
+			badRequestError('You cannot vote for yourself');
+		}
+
 		const totalShares = vote.participants.reduce((sum, s) => sum + s.share, 0);
 		if (totalShares != session.stake) {
 			badRequestError('Total shares must equal session stake');
@@ -110,15 +129,6 @@ export const POST: RequestHandler = async ({ params, request, platform }) => {
 			badRequestError('You have already voted in this session');
 		}
 
-		const voterExists = participants.some((p) => p.id === voterId);
-		if (!voterExists) {
-			badRequestError('Voter is not a participant in this session');
-		}
-
-		if (vote.participants.some((s) => s.id === voterId)) {
-			badRequestError('You cannot vote for yourself');
-		}
-
 		const allShareParticipantsExist = vote.participants.every((s) =>
 			participants.some((p) => p.id === s.id)
 		);