DASSL / ClassDB

An open-source system to let students experiment with relational data
https://dassl.github.io/ClassDB/
Other
8 stars 2 forks source link

Unclear field names in table Student (M) #137

Closed smurthys closed 6 years ago

smurthys commented 6 years ago

Field names lastDDLActivity and lastConnection in table Student are UTC times tamp fields but their names don't communicate the intent.

I propose adding the suffix AtUTC to both field names to clarify the intent. That is, the field names become lastDDLActivityAtUTC and lastConnectionAtUTC.

This issue is related to Issue #49.

smurthys commented 6 years ago

There should be no need to explicitly fix this issue in table Student if Issue #138 is fixed, but the requirement outlined in this issue applies to any new tables that are created to fix Issue #138

smurthys commented 6 years ago

Closed by PR #143