search

blibli-future / athens

0 stars 1 forks source link

Create composite key for attendance #12

Closed riencus closed 7 years ago

riencus commented 7 years ago

Attendance Model only use NIK as its Primary Key---which is false. Since this would means each employee only has one attendance record.

This Pull Request create composite PK using NIK and Date---like what a Attendance record should be. Also change all file affected by this change.

sebastianbagya commented 7 years ago

on second thought, I think I should include a unit test in this pull request < Ok!

riencus commented 7 years ago

@sebastianbagya, Please take look at my newest commit. I need your help

sebastianbagya commented 7 years ago

@arielchristianto, Ok. I will take a look of it after class, its about 30 - 60 min later.