Skip to content

Dev - #1

Open
Nox911PS wants to merge 12 commits into
mainfrom
dev
Open

Dev#1
Nox911PS wants to merge 12 commits into
mainfrom
dev

Conversation

@Nox911PS

@Nox911PS Nox911PS commented Oct 27, 2024

Copy link
Copy Markdown
Owner

Task: https://github.com/AlreadyBored/nodejs-assignments/blob/main/assignments/battleship/assignment.md

Done 28.10.2024 / deadline 28.10.2024

Score: 158 / 188

Bot is not implemented

Notes for Reviewers:

Starting the Application:

To start the app in production mode, run: npm run start
To start the app in development mode, run: npm run start:dev

Login Restrictions

You cannot log in to the game if you already connected

Room Restrictions:

You can create multiple rooms. You cannot add yourself to a room that you created. If you attempt to do so, you will receive an error via WebSocket(I believe it should be implemented on UI).

Room Deletion on Game Start:

When you start the game, all rooms that you created are removed.

User Disconnection and Game Result:

If a user closes their browser page, they are counted as the loser and the connected user is counted as the winner.

Turn-Based Actions:

During the battle, users cannot perform actions when it is not their turn. If a user clicks on the field during the enemy's turn, nothing happens and a WebSocket response is returned. (I believe it should be implemented on UI)

Shooting Restrictions:

During the battle, users cannot shoot at previously targeted cells or the cells surrounding destroyed ships. If a user clicks on such cell, nothing happens and a WebSocket response is returned. (I believe it should be implemented on UI)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants