Skip to content

Game start button enabled#47

Merged
the-only-queen-anna merged 10 commits into
developmentfrom
game-start
May 18, 2026
Merged

Game start button enabled#47
the-only-queen-anna merged 10 commits into
developmentfrom
game-start

Conversation

@the-only-queen-anna
Copy link
Copy Markdown
Collaborator

Context

This pull request enables the function of the game start button. Closes #45

Details

Now, when the game start button is pressed (by the host), the game is started. A message is sent via HTTP with the request for a game start. Then, the server changes the parameter game started to true, and all listening lobby members automatically connect to the lobby's websocket and subscribe to the topic. The host then sends another game-start message after some delay, to make sure all members have already subscribed. The server now broadcasts the first game state object and all members can navigate to their respective game screen.

Changes in the codebase

  • added endpoint for game start messages
  • added message flow
  • added game repository
  • changed some data types
  • only the host can start the game (currently, the game can be started without at least 4 players in the team, for development purposes)

Additional info

N/A

Changes outside the codebase

N/A

- changed color type from String to Color in CardDto
- removed current found parameter from game state
- current found is now computed in game view model
- also removed unused chat messages parameter from game state
- now the flow that the game view model emits is really a game state (no longer the game message)
- corrected a wrong websocket endpoint
- added some mapping functions
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Collaborator

@ad-devel ad-devel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Start button ist jetzt funktional, passt

@the-only-queen-anna the-only-queen-anna merged commit ac82b4d into development May 18, 2026
2 checks passed
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