HappyPantss / real-time-web-2021

This is the repository for Real-Time Web @cmda-minor-web 2020/2021 @cmda-minor-web. In this project I am making my own Unscramble game where you can play real time (multi-user) with your friends! Unscramble the word first, and get the most points! 💬🥇
https://get-scrambled.herokuapp.com/
MIT License
0 stars 1 forks source link

Repository Readme #6

Open bommezijn opened 3 years ago

bommezijn commented 3 years ago

Hi,

Me again, I would like to remind you about Issues #1 where I mentioned several points that I miss in your readme 😢 .

The readme might be a secondary priority right now for you @HappyPantss but don't forget it is one of the criteria (I'll copy the grading table in a comment).

Can't test the rest of the criteria as that part is not available yet to the public.

bommezijn commented 3 years ago
Deficiency Criterion Improvement
Project Your app is working and published on Heroku. Your project is thoroughly documented in the README.md file in your repository. Included are a description of the data-lifecycle, real-time events and external data source used by your app.
Complexity You’ve implemented enough real-time functionality for us to test your comprehension of the subject. A lot of functionality is self-written. You are able to manipulate online examples live.
Client-server interaction By interacting with the app, a user can influence the data model of the server in real time by directly modifying data OR by influencing API requests between server and source. The student has set up the data manipulations.
Data management The server maintains a data model and each client is continuously updated with the correct data.
Multi-user support Multiple clients can connect to the server. Interaction works as expected and is not dependent on the number of clients. You can explain how your app approaches this.
HappyPantss commented 3 years ago

Hi Nathan, thank you for your awesome feedback! My first priority for now is indeed the code, but I have to change this to the readme for now. Thanks for copying the criteria! I'll be working on this as soon as possible!