📝 Description
Briefly list the changes made to the code:
Refactored TimeTableRow.js to instead take in each row as the format it is stored in the Database, requiring TimeTable to no longer need to do pre-processing
Added TimeDuration object to start and end hour, allowing input validation and a UI to select Am / PM
Duration is now calculated from the start and end hour
TimeTable's list of rows is now updated when a start and end time is provided / modified, or new comments are left
Typescript is now used in every file on the src
✔️ Testing
What steps did you take to verify your changes work? These should be clear enough for someone to be able to clone the branch and follow the steps themselves.
Ensured that rows could properly be deleted and added, and appropriate rendering occurred.
Tried wide range of types of start / end times alongside differing decimal values for time.
Ensured that the table correctly triggered callbacks when content was updated.
🏕️ (Optional) Future Work / Notes
Did you notice anything ugly during the course of this ticket? Any bugs, design challenges, or unexpected behavior? Write it down so we can clean it up in a future ticket!
We may need to explore unnecessary re-rendering as I believe that some of the re-rendering here may become unnecessary in some instances (Although react may automatically handle this)
ℹ️ Issue Closes https://github.com/Code-4-Community/breaktime-frontend/issues/26 Closes https://github.com/Code-4-Community/breaktime-frontend/issues/23
📝 Description Briefly list the changes made to the code:
✔️ Testing What steps did you take to verify your changes work? These should be clear enough for someone to be able to clone the branch and follow the steps themselves.
🏕️ (Optional) Future Work / Notes Did you notice anything ugly during the course of this ticket? Any bugs, design challenges, or unexpected behavior? Write it down so we can clean it up in a future ticket!