larssieboy18/discord-uno

An UNO Discord bot hosted on Autocode

discord-uno's Issues

Special card handling on first turn

lines: 64,68
assignees: larssieboy18
labels: enhancement
If a special card is dealt at the first turn, the game should apply that effect to the first player (e.g. skip the player on a skip card, give 2 extra card on a +2 card, etc.)

Line: 28

discord-uno//functions/events/discord/uno/start.js

Lines 25 to 35 in f5cb589

	playerHands[i] = startingDeck.slice(0, 7);
	}
	// TODO Special card handling on first turn
	// lines: 64,68
	// assignees: larssieboy18
	// labels: enhancement
	// If a special card is dealt at the first turn, the game should apply that effect to the first player (e.g. skip the player on a skip card, give 2 extra card on a +2 card, etc.)
	// create discard pile
	let discardPile = [];

cancel button for the game creator

assignees: larssieboy18
lines: 204, 205
labels: enhancement
Add a button to the embed that allows the user to cancel the game.

Line: 228

discord-uno//functions/events/discord/command/start-game.js

Lines 225 to 235 in c3a951f

	}
	];
	// TODO cancel button for the game creator
	// assignees: larssieboy18
	// lines: 204, 205
	// labels: enhancement
	// Add a button to the embed that allows the user to cancel the game.
	/* Sending a message to the channel where the command was used. */
	await responses.update(token, '', embed, [], 'CHANNEL_MESSAGE_WITH_SOURCE');

allAccepted initial message

labels: enhancement
assignees: larssieboy18
lines: 123,132
if everyone has accepted the game, the initial message should be edited to something like: "everyone has accepted the game. It has started in <#channel>"

Line: 117

discord-uno//functions/events/discord/message/button/interaction.js

Lines 114 to 124 in f5cb589

	it will send a message to the user who accepted the game saying that they will be notified when the
	game starts. */
	// TODO allAccepted initial message
	// labels: enhancement
	// assignees: larssieboy18
	// lines: 123,132
	// if everyone has accepted the game, the initial message should be edited to something like: "everyone has accepted the game. It has started in <#channel>"
	let allAccepted
	for (let i = 0; i < fields.length; i++) {

see if there is a more up to date version of this action

Line: 5

discord-uno//.github/workflows/pr-checkboxes.yml

Lines 2 to 12 in 52b76f4

	# TODO: Add a checkbox to the PR template
	# TODO: see if there is a more up to date version of this action
	name: PR Checkboxes
	on:
	pull_request:
	types: [ opened, edited ]

check if selected channel is a text channel

sometimes people can select a voice channel or category, either on purpose or by accident. This should cause the command to fail.
lines: 16, 20
assignees: larssieboy18
labels: enhancement

Line: 23

discord-uno//functions/events/discord/command/start-game.js

Lines 20 to 30 in c3a951f

	/* Getting the channel from the options. */
	let channel = (data.options.find((option) => option.name == `channel`)).value;
	// TODO check if selected channel is a text channel
	// sometimes people can select a voice channel or category, either on purpose or by accident. This should cause the command to fail.
	// lines: 16, 20
	// assignees: larssieboy18
	// labels: enhancement
	// // check if there already is a game in progress inside the same channel
	// let gameInProgress = await kv.get(`unoGame-${guild_id}-${channel}`, false);

Change description of embed when it is not the start of the game anymore

lines 120

Currently, the message shows a message saying that the player that starts the game starts the game. For consequent turns, it should be altered to show that it is someones turn instead of the person starting a game.

discord-uno/functions/events/discord/uno/start.js

Line 120 in b900901

description: `**<@${startingPlayer.id}>** starts the game!`,

allAccepted initial message

if everyone has accepted the game, the initial message should be edited to something like: "everyone has accepted the game. It has started in <#channel>"

---

This issue has been automatically created by todo-actions based on a TODO comment found in functions/events/discord/message/button/interaction.js:117. It will automatically be closed when the TODO comment is removed from the default branch (main).

Dependency Dashboard

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Pending Status Checks

These updates await pending status checks. To force their creation now, click the checkbox below.

• Update dependency js-base64 to v3.7.3
• Update actions/dependency-review-action action to v3

Detected dependencies

github-actions

.github/workflows/PR-labeler.yml

• actions/labeler v4

.github/workflows/auto-assign.yml

• technote-space/assign-author v1

.github/workflows/automatic-translation.yml

• tomsun28/issues-translate-action v2.7

.github/workflows/automerge.yml

• pascalgn/automerge-action v0.15.5

.github/workflows/codeql-analysis.yml

• actions/checkout v3
• github/codeql-action v2
• github/codeql-action v2
• github/codeql-action v2

.github/workflows/dependency-review.yml

• actions/checkout v3
• actions/dependency-review-action v2

.github/workflows/issue-labeler.yml

• github/issue-labeler v2.5

.github/workflows/pr-checkboxes.yml

• loxygenK/taskonfirm v1.0.0

.github/workflows/release-drafter.yml

• release-drafter/release-drafter v5

.github/workflows/semgrep.yml

• actions/checkout v3

.github/workflows/stale.yml

• actions/stale v6

.github/workflows/todo.yml

npm

package.json

• js-base64 3.7.2

---

• Check this box to trigger a request for Renovate to run again on this repository

Create Pull Request template

Special card handling on first turn

If a special card is dealt at the first turn, the game should apply that effect to the first player (e.g. skip the player on a skip card, give 2 extra card on a +2 card, etc.)

discord-uno/functions/events/discord/uno/start.js

Line 34 in c35ce2a

// create discard pile

  // create discard pile
  let discardPile = [];

  // add first card to discard pile
  discardPile.push(startingDeck.pop());

cancel button for the game creator

Add a button to the embed that allows the user to cancel the game.

---

This issue has been automatically created by todo-actions based on a TODO comment found in functions/events/discord/command/start-game.js:199. It will automatically be closed when the TODO comment is removed from the default branch (main).

remove the double message

Decide where the message should be sent and remove the other message.

---

This issue has been automatically created by todo-actions based on a TODO comment found in functions/events/discord/command/start-game.js:206. It will automatically be closed when the TODO comment is removed from the default branch (main).

remove test data before pushing to master

discord-uno/functions/events/discord/uno/start.js

Line 7 in 6f94f8e

 // debugging - provide test data while debugging
 let context = {
   params: {
     game: {
       gameChannel: `976400262677803018`,
       players: [
         {
           id: `119473151913623552`
         },
         {
           id: `793188304764928023`
         }
       ]
     },
     event: {
       guild_id: `975789410186567730`,
     },
   },
 }

cancel button for the game creator

assignees: larssieboy18
lines: 204, 205
labels: enhancement
Add a button to the embed that allows the user to cancel the game.

Line: 228

discord-uno//functions/events/discord/command/start-game.js

Lines 225 to 235 in f5cb589

	}
	];
	// TODO cancel button for the game creator
	// assignees: larssieboy18
	// lines: 204, 205
	// labels: enhancement
	// Add a button to the embed that allows the user to cancel the game.
	/* Sending a message to the channel where the command was used. */
	await responses.update(token, '', embed, [], 'CHANNEL_MESSAGE_WITH_SOURCE');

Remove test data from `all.js`

Remove all test data, i.e. the one listed in the first 112 lines of this file, from all.js.
e.g. replace hello with event and remove the hello variable.
replace eventt with event and remove the eventt variable.

Line: 117

discord-uno//functions/events/discord/message/button/interaction/all.js

Lines 1 to 117 in f8be554

	let hello = {
	event: {
	"version": 1,
	"user": {
	"username": "aCoolguy63",
	"public_flags": 0,
	"id": "793188304764928023",
	"discriminator": "8503",
	"avatar_decoration": null,
	"avatar": "068c83a0ed3de3c445cc69bb5b1c6a5b"
	},
	"type": 3,
	"token": "aW50ZXJhY3Rpb246MTAyMjU3NjMwNzcwOTY3NzcxMDpUNDFLbnk4ODJqN3ZkS2xpRWdjeld5WG1ocjdpMEJrUFZXRlQ5eEt2N0J1eFdsSG4xVldGUDhRb1J4UXNRcjlKajNFSDZ4N0xxYnZTYU8xbUZwSVc0MWE3a2JZOVVKb1hHUlh1NmtDd2xMR1hkMm1QcW41cHdiekF5RjBGZlZNeA",
	"message": {
	"type": 0,
	"tts": false,
	"timestamp": "2022-09-22T18:33:03.071000+00:00",
	"pinned": false,
	"mentions": [],
	"mention_roles": [],
	"mention_everyone": false,
	"id": "1022576277359693965",
	"flags": 0,
	"embeds": [
	{
	"type": "rich",
	"title": "UNO",
	"fields": [
	{
	"value": "<@119473151913623552>",
	"name": "Invited by",
	"inline": true
	},
	{
	"value": "<t:1663871577:f>",
	"name": "Invited at",
	"inline": true
	},
	{
	"value": "**UNO BLUE & GREEN + CONFIG** (*ID*: `975789410186567730`)",
	"name": "Server",
	"inline": true
	},
	{
	"value": "[Click here](https://discordapp.com/channels/975789410186567730/976400262677803018/1022576261450702918)",
	"name": "Invite message",
	"inline": true
	},
	{
	"value": "❌",
	"name": "Accepted by you?",
	"inline": true
	}
	],
	"description": "Hello <@793188304764928023>! You have been invited to play UNO.\nDo you accept?",
	"color": 3092790
	}
	],
	"edited_timestamp": null,
	"content": "",
	"components": [
	{
	"type": 1,
	"components": [
	{
	"type": 2,
	"style": 3,
	"label": "Accept",
	"emoji": {
	"name": "acheck",
	"id": "984488658658807818",
	"animated": true
	},
	"custom_id": "accept-uno-invite"
	},
	{
	"type": 2,
	"style": 4,
	"label": "Deny",
	"emoji": {
	"name": "across",
	"id": "984488679173140520",
	"animated": true
	},
	"custom_id": "deny-uno-invite"
	}
	]
	}
	],
	"channel_id": "984497481821847602",
	"author": {
	"username": "UNO test bot",
	"public_flags": 0,
	"id": "976400332299063366",
	"discriminator": "0545",
	"bot": true,
	"avatar_decoration": null,
	"avatar": null
	},
	"attachments": []
	},
	"locale": "en-GB",
	"id": "1022576307709677710",
	"data": {
	"custom_id": "deny-uno-invite",
	"component_type": 2
	},
	"channel_id": "984497481821847602",
	"application_id": "976400332299063366",
	"received_at": "2022-09-22T18:33:10.391Z"
	}
	}
	// // get event info
	// let { event } = context.params
	// TODO Remove test data from `all.js`

Game turn

• Notify everyone that new turn has started
• Get data from previous turn
• Check if correct player played a card
• Check if card is valid
• Check if player has UNO
• Check if player called UNO
• Save turn info to kv
• Notify everyone that card has been played
• Check if someone calls the fact that player failed to call UNO

check if selected channel is a text channel

sometimes people can select a voice channel or category, either on purpose or by accident. This should cause the command to fail.

discord-uno/functions/events/discord/command/start-game.js

Line 20 in ac1391a

let channel = (data.options.find((option) => option.name == `channel`)).value;

let { event } = context.params,
  { member, data, token, guild_id, received_at } = event;

/* Getting the channel from the options. */
let channel = (data.options.find((option) => option.name == `channel`)).value;

TODO - merge Github action files

Merge all (compatible) action files into 1 .yml file to limit the amount of separate files

Feel free to contribute to this.

remove the double message

labels: bug
assignees: larssieboy18
lines: 214
Decide where the message should be sent and remove the other message.
Sending a message to the channel where the game will be played. */

Line: 237

discord-uno//functions/events/discord/command/start-game.js

Lines 234 to 244 in c3a951f

	/* Sending a message to the channel where the command was used. */
	await responses.update(token, '', embed, [], 'CHANNEL_MESSAGE_WITH_SOURCE');
	// TODO remove the double message
	// labels: bug
	// assignees: larssieboy18
	// lines: 214
	// Decide where the message should be sent and remove the other message.
	/* Sending a message to the channel where the game will be played. */
	await messages.create(channel, '', embed, []);

allAccepted initial message

labels: enhancement
assignees: larssieboy18
lines: 123,132
if everyone has accepted the game, the initial message should be edited to something like: "everyone has accepted the game. It has started in <#channel>"

Line: 117

discord-uno//functions/events/discord/message/button/interaction.js

Lines 114 to 124 in c3a951f

	it will send a message to the user who accepted the game saying that they will be notified when the
	game starts. */
	// TODO allAccepted initial message
	// labels: enhancement
	// assignees: larssieboy18
	// lines: 123,132
	// if everyone has accepted the game, the initial message should be edited to something like: "everyone has accepted the game. It has started in <#channel>"
	let allAccepted
	for (let i = 0; i < fields.length; i++) {

Move all text to locale files to ease (future) translation

Change description of embed when it is not the start of the game anymore

lines: 120
labels: development
assignees: larssieboy18
Currently, the message shows a message saying that the player that starts the game starts the game. For consequent turns, it should be altered to show that it is someones turn instead of the person starting a game.

Line: 131

discord-uno//functions/events/discord/uno/start.js

Lines 128 to 138 in f5cb589

	},
	], []);
	// TODO Change description of embed when it is not the start of the game anymore
	// lines: 120
	// labels: development
	// assignees: larssieboy18
	// Currently, the message shows a message saying that the player that starts the game starts the game. For consequent turns, it should be altered to show that it is someones turn instead of the person starting a game.
	// did an error occur?
	return false

Change description of embed when it is not the start of the game anymore

lines: 120
labels: development
assignees: larssieboy18
Currently, the message shows a message saying that the player that starts the game starts the game. For consequent turns, it should be altered to show that it is someones turn instead of the person starting a game.

Line: 131

discord-uno//functions/events/discord/uno/start.js

Lines 128 to 138 in c3a951f

	},
	], []);
	// TODO Change description of embed when it is not the start of the game anymore
	// lines: 120
	// labels: development
	// assignees: larssieboy18
	// Currently, the message shows a message saying that the player that starts the game starts the game. For consequent turns, it should be altered to show that it is someones turn instead of the person starting a game.
	// did an error occur?
	return false

allAccepted initial message

if everyone has accepted the game, the initial message should be edited to something like: "everyone has accepted the game. It has started in <#channel>"

---

This issue has been automatically created by todo-actions based on a TODO comment found in functions/events/discord/message/button/interaction.js:117. It will automatically be closed when the TODO comment is removed from the default branch (main).

Move all text to language files

Make sure that all plain text (and variables) are included in the /lang/en.json file so that translation is possible.

check if selected channel is a text channel

sometimes people can select a voice channel or category, either on purpose or by accident. This should cause the command to fail.
lines: 16, 20
assignees: larssieboy18
labels: enhancement

Line: 23

discord-uno//functions/events/discord/command/start-game.js

Lines 20 to 30 in f5cb589

	/* Getting the channel from the options. */
	let channel = (data.options.find((option) => option.name == `channel`)).value;
	// TODO check if selected channel is a text channel
	// sometimes people can select a voice channel or category, either on purpose or by accident. This should cause the command to fail.
	// lines: 16, 20
	// assignees: larssieboy18
	// labels: enhancement
	// // check if there already is a game in progress inside the same channel
	// let gameInProgress = await kv.get(`unoGame-${guild_id}-${channel}`, false);

Special card handling on first turn

lines: 64,68
assignees: larssieboy18
labels: enhancement
If a special card is dealt at the first turn, the game should apply that effect to the first player (e.g. skip the player on a skip card, give 2 extra card on a +2 card, etc.)

Line: 28

discord-uno//functions/events/discord/uno/start.js

Lines 25 to 35 in c3a951f

	playerHands[i] = startingDeck.slice(0, 7);
	}
	// TODO Special card handling on first turn
	// lines: 64,68
	// assignees: larssieboy18
	// labels: enhancement
	// If a special card is dealt at the first turn, the game should apply that effect to the first player (e.g. skip the player on a skip card, give 2 extra card on a +2 card, etc.)
	// create discard pile
	let discardPile = [];

Automatic close label does not work

close-label.yml contains an error, where a - is used instead of _

add buttons to starting message

labels: enhancement
assignees: larssieboy18
lines: 86,105

Line: 74

discord-uno//functions/events/discord/uno/start.js

Lines 71 to 81 in c3a951f

	// save game to kv
	await kv.set(`UNO-${guild_id}-${gameChannel}`, game, 604800 /* a week */);
	// TODO add buttons to starting message
	// labels: enhancement
	// assignees: larssieboy18
	// lines: 86,105
	// send starting message to game channel
	await messages.create(gameChannel, '', [
	{

add buttons to starting message

Add buttons to starting message

discord-uno/functions/events/discord/uno/start.js

Line 86 in 04edf9c

// send starting message to game channel

 // send starting message to game channel
 let startingMessage = await messages.create(gameChannel, '', [
   {
     title: "UNO",
     description: `**<@${startingPlayer.id}>** starts the game!`,
     color: 0x00ff00,
     fields: [
       {
         name: "Current Card",
         value: `${discardPile[0]}`,
         inline: true,
       },
       {
         name: "Current Player",
         value: `${startingPlayer.name}`,
         inline: true,
       },
     ],
   }
 ], []);

add buttons to starting message

labels: enhancement
assignees: larssieboy18
lines: 86,105

Line: 74

discord-uno//functions/events/discord/uno/start.js

Lines 71 to 81 in f5cb589

	// save game to kv
	await kv.set(`UNO-${guild_id}-${gameChannel}`, game, 604800 /* a week */);
	// TODO add buttons to starting message
	// labels: enhancement
	// assignees: larssieboy18
	// lines: 86,105
	// send starting message to game channel
	await messages.create(gameChannel, '', [
	{

Add a checkbox to the PR template

Line: 4

discord-uno//.github/workflows/pr-checkboxes.yml

Lines 1 to 11 in 52b76f4

	# https://github.com/loxygenK/taskonfirm
	# TODO: Add a checkbox to the PR template
	# TODO: see if there is a more up to date version of this action
	name: PR Checkboxes
	on:
	pull_request:
	types: [ opened, edited ]