Skip to content

[EXTERNAL] make-your-game:score-handling WebSocket package is not needed#2894

Merged
HarryVasanth merged 1 commit into01-edu:masterfrom
sadiqui:patch-11
Mar 24, 2025
Merged

[EXTERNAL] make-your-game:score-handling WebSocket package is not needed#2894
HarryVasanth merged 1 commit into01-edu:masterfrom
sadiqui:patch-11

Conversation

@sadiqui
Copy link
Copy Markdown
Contributor

@sadiqui sadiqui commented Feb 28, 2025

The Gorilla WebSocket package is not needed for this project. WebSockets are typically used for real-time bidirectional communication, such as live chat or multiplayer games. However, since only one player interacts with the game at a time, WebSockets are unnecessary.

For this project, a Go API service using HTTP endpoints is sufficient to handle the scoreboard. The player’s score will be sent via a POST request, and the server will store it in a JSON file. A GET request will retrieve the stored scores when needed. This is explicitly stated in the project instructions:

For this project you must implement:
- **scoreboards**
- **go API service** that allows saving the data from the game in JSON format
- The API should accept POST and GET requests from the client side for the scoreboard data

The implementation will rely only on two standard Go packages:

This project is designed to help students learn how to build simple APIs using Go’s standard library:

This project will help you learn about:
- Creating and using an API
- POST requests
- GET requests
- JSON
- Sorting algorithms

Additionally, the audit ensures that only allowed packages are used:

###### Has the requirement for the allowed packages been respected? (Reminder for this project: only [standard packages](https://golang.org/pkg/))

@HarryVasanth HarryVasanth merged commit 045cc44 into 01-edu:master Mar 24, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗂️ docs Documentation 🤸 skip-check-links Skip check links

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants