fix: response & resolution time stats

discord-tickets · Feb 10, 2023 · ced14ce · ced14ce
1 parent a953308
commit ced14ce
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 8 deletions.
diff --git a/src/lib/tickets/manager.js b/src/lib/tickets/manager.js
@@ -406,9 +406,17 @@ module.exports = class TicketManager {
 		const statsCacheKey = `cache/category-stats/${categoryId}`;
 		let stats = await this.client.keyv.get(statsCacheKey);
 		if (!stats) {
-			const { tickets } = await this.client.prisma.category.findUnique({
-				select: { tickets: { where: { open: false } } },
-				where: { id: categoryId },
+			const tickets = await this.client.prisma.ticket.findMany({
+				select: {
+					closedAt: true,
+					createdAt: true,
+					firstResponseAt: true,
+				},
+				where: {
+					categoryId: category.id,
+					firstResponseAt: { not: null },
+					open: false,
+				},
 			});
 			stats = {
 				avgResolutionTime: ms(tickets.reduce((total, ticket) => total + (ticket.closedAt - ticket.createdAt), 0) ?? 1 / tickets.length),

diff --git a/src/listeners/client/ready.js b/src/listeners/client/ready.js
@@ -33,11 +33,12 @@ module.exports = class extends Listener {
 			if (!cached) {
 				const tickets = await client.prisma.ticket.findMany({
 					select: {
+						closedAt: true,
 						createdAt: true,
 						firstResponseAt: true,
 					},
 				});
-				const closedTickets = tickets.filter(t => t.closedAt);
+				const closedTickets = tickets.filter(t => t.firstResponseAt && t.closedAt);
 				cached = {
 					avgResolutionTime: ms(closedTickets.reduce((total, ticket) => total + (ticket.closedAt - ticket.createdAt), 0) ?? 1 / closedTickets.length),
 					avgResponseTime: ms(closedTickets.reduce((total, ticket) => total + (ticket.firstResponseAt - ticket.createdAt), 0) ?? 1 / closedTickets.length),

diff --git a/src/routes/api/admin/guilds/[guild]/categories/index.js b/src/routes/api/admin/guilds/[guild]/categories/index.js
@@ -29,7 +29,7 @@ module.exports.get = fastify => ({
 			where: { id: req.params.guild },
 		});
 		categories = categories.map(c => {
-			const closedTickets = c.tickets.filter(t => t.closedAt);
+			const closedTickets = c.tickets.filter(t => t.firstResponseAt && t.closedAt);
 			c = {
 				...c,
 				stats: {

diff --git a/src/routes/api/admin/guilds/[guild]/index.js b/src/routes/api/admin/guilds/[guild]/index.js
@@ -23,12 +23,13 @@ module.exports.get = fastify => ({
 			});
 			const tickets = await client.prisma.ticket.findMany({
 				select: {
+					closedAt: true,
 					createdAt: true,
 					firstResponseAt: true,
 				},
 				where: { guildId: id },
 			});
-			const closedTickets = tickets.filter(t => t.closedAt);
+			const closedTickets = tickets.filter(t => t.firstResponseAt && t.closedAt);
 			cached = {
 				createdAt: settings.createdAt,
 				id: guild.id,
@@ -46,7 +47,7 @@ module.exports.get = fastify => ({
 					tickets: tickets.length,
 				},
 			};
-			await client.keyv.set(cacheKey, cached, ms('5m'));
+			// await client.keyv.set(cacheKey, cached, ms('5m'));
 		}
 
 		return cached;

diff --git a/src/routes/api/client.js b/src/routes/api/client.js
@@ -10,11 +10,12 @@ module.exports.get = () => ({
 		if (!cached) {
 			const tickets = await client.prisma.ticket.findMany({
 				select: {
+					closedAt: true,
 					createdAt: true,
 					firstResponseAt: true,
 				},
 			});
-			const closedTickets = tickets.filter(t => t.closedAt);
+			const closedTickets = tickets.filter(t => t.firstResponseAt && t.closedAt);
 			const users = await client.prisma.user.findMany({ select: { messageCount: true } });
 			cached = {
 				avatar: client.user.avatarURL(),