pittktp / ktp-react

Migration of the current Angular 6 based website to React
2 stars 1 forks source link

Taking Attendance Not Working #33

Closed StevenMonty closed 3 years ago

StevenMonty commented 3 years ago

When taking attendance, clicking "Not here" on a brother removes another random brother from the list but not the one clicked. While revisiting this feature, I think it would be more user friendly to replace "Not Here" with "Here" so you don't have to try and identify who is missing rather than who is present.

ZvontyFlugence commented 3 years ago

Most recent push fixed this issue, still needs to be deployed. I agree that it would be more user friendly to use 'Here' instead of 'Not Here', however, that would require a complete restructuring of how the attendance form works and how the API handles taking attendance. Due to this, I will open a new issue as an enhancement to tackle this.