Only the original player can interact

[Taz03]

Right now anyone can interact with a 2048 game, make it so that only the user who started the game can interact with it.

Implementation wise, pass the id of user who started the game to button and check weather it is still the same user interacting, otherwise just ignore the interaction or send a ephemeral message saying "You can't interact with this game" or similar.

Add user id here after command name

TJ-Plays/app/src/main/java/com/togetherjava/tjplays/listeners/commands/Game2048Command.java

Lines 68 to 76 in e226b5b

	private MessageCreateData gameMessage(Renderer2048 gameRenderer) {
	Button resetButton = Button.success(COMMAND_NAME + " reset", Emoji.fromUnicode("🔃"));
	Button upButton = Button.primary(COMMAND_NAME + " up", Emoji.fromUnicode("⬆️"));
	Button deleteButton = Button.danger(COMMAND_NAME + " delete", Emoji.fromUnicode("🗑️"));
	Button leftButton = Button.primary(COMMAND_NAME + " left", Emoji.fromUnicode("⬅️"));
	Button downButton = Button.primary(COMMAND_NAME + " down", Emoji.fromUnicode("⬇️"));
	Button rightButton = Button.primary(COMMAND_NAME + " right", Emoji.fromUnicode("➡️"));

check user id this method after checking command name

TJ-Plays/app/src/main/java/com/togetherjava/tjplays/listeners/commands/Game2048Command.java

Line 44 in e226b5b

public void onButtonInteraction(ButtonInteractionEvent event) {

[ooplesoft]

getUser method available from SlashCommand package. https://ci.dv8tion.net/job/JDA/javadoc/net/dv8tion/jda/api/events/interaction/SlashCommandEvent.html

Suggestion: save Initial ID of user that creates game via slash command, then read ID of user initiating action and if they are not equal, ignore the action/ return a message.

[ooplesoft]

@Zabuzard or @Taz03 can you assign this to me so I can try my hand at it? I think you just need to save the user ID on the initial slash command event and then do a check on the button interaction event to see if UserId is the same

[Taz03]

@ooplesoft sure, assigned u. Yes thats the correct approach

[ooplesoft]

Issue was fixed. Please reference my forked commit. Preview screenshot of message is here:

[Taz03]

@ooplesoft u will have to make a pr on this repo, which will then be reviewed and get merged

[ooplesoft]

Thanks for letting me know! This is my first time doing this :)
#9