ashesi-SE / news-on-time

A virtual notice board for Ashesi University College
0 stars 0 forks source link

Master to Stable merge #46

Closed aycarl closed 9 years ago

toddwseattle commented 9 years ago

Final Submission Grade

Team: Once a Week (news on time) Score: 18/25

Overall

adjusted based on finding pull request, instructions to run. Documentation looks pretty good. good instructions on how to clone and build the applciation. Source code is hard to follow and no pull request as noted. App looked good; but some database connection problem meant it wasn't really functional which hurt.

Breakdown

Score Possible Description Comments
4 5 Wiki Complete. Makes sense to an independent observer. Naviagable to understand purpose and location of the application generally makes sense. missing a few elements of the final assignment like user instructions for how to run?
5 10 Deployed Application Runs as Expected. I can enter data and use it without a crash or strange error message. Can get access to it. Design makes sense. Data makes sense running on 11/11 I could not get any of the entered data to save; with our without a picture :( half credit.
4 5 Source code and source code control: git used effectively, pull request exists, comments and organization of source are understandable to a new user. No pull request as of this writing. source is a little hard to follow without more information. no real comments to speak of.
5 5 Instructions let users build and run application successfully. Could be cloned locally these are pretty clear in the readme. don't understand branching--why is everything in iteration 3?