cs102-project / LabConnect

LabConnect facilitates communication between students, TA's, tutors, and instructors.
http://labconnect.me
GNU Affero General Public License v3.0
6 stars 0 forks source link

Pre-checkpoint cleanup #15

Closed berkan-sahin closed 3 years ago

berkan-sahin commented 3 years ago

There are only a few hours left before the first checkpoint. We are (un)officially in a feature freeze and in a need to clean up the code prior to submission.

TODO

berkan-sahin commented 3 years ago

@whymeeverytime could you post the announcement made by A.Dayanık here?

friberk commented 3 years ago

I haven't seen any problems with variable naming

whymeeverytime commented 3 years ago

Moodle

whymeeverytime commented 3 years ago

I don't think there is an unnecessary comment. I think we can check that box in the list.

berkan-sahin commented 3 years ago

@vedattt @Apu322 guys, can you check the documentation? (You can credit Vedat where credit's due) Also we need to write everyone's contribution

berkan-sahin commented 3 years ago
whymeeverytime commented 3 years ago

For the Current Status: Testing logic (Unit test and style checker) was implemented, as well as the general I/O infrastructure for files needing persistent storage. Assignment, submission, and user type model classes were also implemented.

whymeeverytime commented 3 years ago

For the Further Development: The only "business logic" needing implementation is the Live Session logic. Once unit classes are complete, Repository classes for DB queries must be written (most of the heavy lifting is already done by Spring Data), and a REST API should be provided for manipulating the model classes. After this, the frontend code will be written based on the UI report and the REST API provided by the backend.

berkan-sahin commented 3 years ago

The status (and further dev.) seems complete to me

berkan-sahin commented 3 years ago

Created PR #16 to merge the backend code until this point to master

berkan-sahin commented 3 years ago

The docs seem complete to me. If someone else can confirm, we can move on with the merge.

berkan-sahin commented 3 years ago

Proceeding with release. Will write later once submission zip is ready

berkan-sahin commented 3 years ago

@friberk will submit the code and close the issue

friberk commented 3 years ago

God bless G2C 🙏🏻