nVarap / blog

Apache License 2.0
0 stars 3 forks source link

Final Review Ticket #18

Open nVarap opened 1 year ago

nVarap commented 1 year ago

Final Review Ticket -- Peer and Self Grade

Deployed Website: DON'T LEAVE ME (Create an account! Find love! ❤️❤️❤️)

Part 1: Live Review with Student Crossovers (7 Points Total)

Item Self-Grade First Person (Samarth) Second Person (Ranauk) Reasoning Avg
Runtime (binary criteria below, 3 points + bonus) n/a n/a n/a n/a n/a
Observe Team project quality, runs on cross domain, performs all evaluations without crashing. 1 1 1 Runs on a deployed domain, operates completely correctly without crashing. 1
Observe, Individual UI contains inputs that are validated for garbage, data cleaned (ie validate DOB or Password) 1 1 1 The UI contains inputs, and the functions throw error when invalid emails are inputted. Validate functions also exist to ensure continuation only occurs if the function is passed 1
Observe, Individual UI contains actions for Create, Read 1 1 1 UI Actively has textbooks that can be used to create, and text boxes that use promises to fill out certain text boxes by reading from the Database 1
Observe, Individual UI contains actions for Update, Delete (above requirement) 0 0 0 I did not have any actual update or delete calls. 0
Code Review (binary criteria below, 4 points + bonus) n/a n/a n/a n/a n/a
Individual code quality, review and observe comments in every function, class, or method reviewed. 1 1 1 Our code contains concise comments by every function that allows for easy reading. Our frontend also has comments that allow for our backend devs (myself included) to know what each element is for without doing something that could break the website. 1
BE review, observe if Individual API handles error conditions (ie duplicate user id) 1 1 1 In the creation portion of our site, if you have an email that is already in the database, the function will throw an error. This allows for greater control over what enters and exits our database. Garbage in, garbage out. 1
BE review, observe if Individual API contains GET and POST methods 1 1 1 Our code operates using multiple functions which each store an individual request to our database. As a result, we contain GET methods. We also have authHandlers which handles authentication and POST methods to our database. 1
BE review, observe that Individual code contains database operations that support Create, Read 1 1 1 My code has many calls which contain the database operations GET and POST, or Create and Read. This is then supported by our database, which can actively test write and read methods, and definitely supports it. 1
BE review, observe that Individual code contains database operations that support Update, Delete (above requirement) 1 1 1 I earn this extra point as our code contains database operations that can support Update and Delete function, and our database similarly supports these types of connections. Therefore, I earn this point 1

Part 2: Silent Review by providing Review Ticket to Teacher (4 Points Total)

Total: 8/7 and 4/4 = 12/11 total