Skip to content

Commit 1238b5c

Browse files
committed
added user mention to strike giving and removing
1 parent 70ef67b commit 1238b5c

File tree

3 files changed

+23
-9
lines changed

3 files changed

+23
-9
lines changed

src/commands/moderation/playerModeration/giveStrike.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,10 +50,11 @@ export default {
5050
reference: referenceChannel.name ?? 'unknown channel',
5151
})
5252

53+
const member = await interaction.guild!.members.fetch(user.id)
5354
// log usage
5455
const embed = createEmbedType(
5556
`#${strikeId} - STRIKE GIVEN`,
56-
'desc.',
57+
`<@${user.id}>`,
5758
null, // default
5859
[
5960
{ name: `Amount`, value: `${amount}`, inline: true },
@@ -67,7 +68,7 @@ export default {
6768
null,
6869
`${blame}`,
6970
)
70-
await logStrike('add_strike', embed)
71+
await logStrike('add_strike', embed, undefined, `<@${user.id ?? 1234}>`)
7172

7273
await interaction.editReply(
7374
`User ${username} given ${amount} strikes ${reason == 'No reason provided' ? `for ${reason}` : ``}`,

src/commands/moderation/playerModeration/removeStrike.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,12 @@ export default {
3434
null,
3535
`${blame}`,
3636
)
37-
await logStrike('remove_strike', embed)
37+
await logStrike(
38+
'remove_strike',
39+
embed,
40+
undefined,
41+
`<@${strikeInfo.user_id ?? 1234}>`,
42+
)
3843
} catch (err: any) {
3944
console.error(err)
4045
}

src/utils/logCommandUse.ts

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -110,12 +110,16 @@ export abstract class Embed {
110110

111111
export class CommandFactory extends Embed {
112112
// build child instances
113-
static build(commandType: string, id?: number) {
113+
static build(commandType: string, id?: number, description?: string) {
114114
switch (commandType) {
115115
case 'add_strike':
116-
return new AddStrike()
116+
const addStrike = new AddStrike()
117+
addStrike.description = description ?? ' '
118+
return addStrike
117119
case 'remove_strike':
118-
return new RemoveStrike()
120+
const removeStrike = new RemoveStrike()
121+
removeStrike.description = description ?? ' '
122+
return removeStrike
119123
case 'general':
120124
return new General()
121125
case 'room':
@@ -156,11 +160,15 @@ export class AddStrike extends CommandFactory {
156160
// distilled process to log an EmbedType object
157161
// @parameter
158162
// type - choose from a list of embed types ['add_strike', 'remove_strike', 'general', 'room']
159-
export async function logStrike(type: string, embed: EmbedType, id?: number) {
163+
export async function logStrike(
164+
type: string,
165+
embed: EmbedType,
166+
id?: number,
167+
desc?: string,
168+
) {
160169
// build strike child class using type as parameter in factory
161-
const strike = CommandFactory.build(type, id)
170+
const strike = CommandFactory.build(type, id, desc)
162171
if (!strike) return
163-
164172
// build embed using info from an EmbedType object
165173
strike.setAll(embed)
166174
strike.createEmbed()

0 commit comments

Comments
 (0)