Added a embed style for all command.

[BadEnd777]

Description

Added a embed style for the report command.

Changes

• Deleted

await channel.send(
  [
    `Reported message: https://discord.com/channels/${interaction.guild.id}/${interaction.channelId}/${interaction.targetId} (reported ${messageReportCount} times)`,
    `Message author: ${member.user} (reported ${reporteeCount} times)`,
    `Reported by: ${interaction.user} (sent ${reporterCount} reports)`,
  ].join('\n'),
)

• Added

const messagelink = `https://discord.com/channels/${interaction.guild.id}/${interaction.channelId}/${interaction.targetId}`

// TODO: Add a button to delete the message and ban the user (Mod only)
const embed = {
  title: 'Reported message',
  url: messagelink,
  color: 0xff0000,
  thumbnail: {
    url: member.user.displayAvatarURL(),
  },
  fields: [
    { name: 'Message author', value: member.toString(), inline: true },
    { name: 'Reported by', value: interaction.user.toString(), inline: true },
    {
      name: 'Reported message',
      value: `\`\`\`${message.content}\`\`\`\n[Jump to message](${messagelink})`,
    },
  ],
  timestamp: new Date(),
}

await channel.send({ embeds: [Object.assign(embed)] })

[buskun]

Can we separate the embed template creation to another function accepting required information?

const getMessageReportingEmbed = (...) => ({ /* Embed object here */ })

[BadEnd777]

yap, it can be done
but normally, we create a new embed for each command because the data we need is not the same
but there are some that can be used as a template, such as
image command that will only have an image and description

Ex: template of image command

cosnt ImageEmbed = (url: string, description: string) => ({
  description,
  image: { url },
})

Ex: usage

await channel.send({
  embeds: [ImageEmbed("https://i.imgur.com/...", "This is a cat")],
});

but actually the discord.js lib already has EmbedBuilder to use, but I didn't use it because I think it's not cool XD.

[dtinth]

please add some screenshot so we see the end result krubb 🙏

[dtinth]

meanwhile, i just merged #37 because the implementation is simple and PR is very small scope and the PR author also provided a screenshot. but there are other valid improvements in this PR, please rebase krub 🙏

[buskun]

yap, it can be done but normally, we create a new embed for each command because the data we need is not the same but there are some that can be used as a template, such as image command that will only have an image and description

Ex: template of image command

cosnt ImageEmbed = (url: string, description: string) => ({
  description,
  image: { url },
})

Ex: usage

await channel.send({
  embeds: [ImageEmbed("https://i.imgur.com/...", "This is a cat")],
});

but actually the discord.js lib already has EmbedBuilder to use, but I didn't use it because I think it's not cool XD.

My purpose is not for reusability for this matter, it is more toward readability and testing (in the future).
Also, I am fine with both builder and raw one, but if we are using raw js object one, please also specify type for the embed variable.

[beam41]

project actually use node 18, you might want to change this

[buskun]

Actually, in package.json, it specifies both v16 and v18.

[BadEnd777]

Yap v16 and v18, but i'm use v19 is default.
I and someone must change node version to develop.

Thank you

[leomotors]

Change to nvm use. It will auto detect the version in .nvmrc

[BadEnd777]

please add some screenshot so we see the end result krubb 🙏

[BadEnd777]

yap, it can be done but normally, we create a new embed for each command because the data we need is not the same but there are some that can be used as a template, such as image command that will only have an image and description
Ex: template of image command

cosnt ImageEmbed = (url: string, description: string) => ({
  description,
  image: { url },
})

Ex: usage

await channel.send({
  embeds: [ImageEmbed("https://i.imgur.com/...", "This is a cat")],
});

but actually the discord.js lib already has EmbedBuilder to use, but I didn't use it because I think it's not cool XD.

My purpose is not for reusability for this matter, it is more toward readability and testing (in the future). Also, I am fine with both builder and raw one, but if we are using raw js object one, please also specify type for the embed variable.

I will try. Thank you.

[beam41]

This revert #33 please change back

[BadEnd777]

Back then.

[leomotors]

Please run pnpm format

[BadEnd777]

Please run pnpm format

all done. maybe.