Note for Peer Reviewers: Rav and Jay are working on the gallery component, but havent finished yet, so we've got all the db stuff working, ran tests for the galleryHelper and the gallery function and they both work. So once our code is merged with the Dev branch, they should be able to pull that down and use our work to connect their component to the db.
Thanks! D n A
Definition of Done
For every submission to the codebase, check that it meets the DoD below.
[ ] Code fulfils the user story and scenarios set out in the github task and UI prototype.
[ ] The feature has been thoroughly manually tested. (Ask other devs to give some fresh eyes on this as you go.)
[ ] All tests run and passed from terminal. Any broken tests fixed, not disabled or skipped.
npm test
[ ] Tests added or updated where appropriate. (Be pragmatic about adding new tests, think; how critical is this module or function? If this function broke due to changes, how would it affect the user experience?)
[ ] eslint run and passes. Any issues fixed.
npm run lint
[ ] PR created in Github.
[ ] Two peer reviews from other devs.
[ ] One peer review from one of the teachers.
[ ] After all the peer review comments are addressed, the branch is merged to main!
Note for Peer Reviewers: Rav and Jay are working on the gallery component, but havent finished yet, so we've got all the db stuff working, ran tests for the galleryHelper and the gallery function and they both work. So once our code is merged with the Dev branch, they should be able to pull that down and use our work to connect their component to the db. Thanks! D n A
Definition of Done
For every submission to the codebase, check that it meets the DoD below.