DeltaO3 / UWAttend---3200_27

Repository for CITS3200 group 27
1 stars 0 forks source link

Sprint 3 - Database Schema cleanup #103

Open DeltaO3 opened 1 week ago

DeltaO3 commented 1 week ago

The client noted during the demonstration that they want the database not to use numbers to represent what is effectively strings (e.g. use admin,coordinator, facilitator instead of 1 2 3, use true or false instead of 0 and 1 (i disagree here), etc.)

The client also made note of potentially not needing the active column since we can just use dates.

Any changes should be reflected across the repository - change any database functions to reflect this, use a global search to edit all calls to these functions to reflect these changes. Ideally include testdb.py in these changes (for ease of testing), but note that file will eventually be removed from the repo in the final week

livvy10 commented 1 week ago

Another change we can do here is changing the uwaID field to string rather than number. Because if the number is 00012222, then the database will store it as 12222, and if we are displaying it on the webapp and using it for emails we would need it to be accurate.

Don't know how I missed that haha

DeltaO3 commented 1 week ago

Very true! will we need to pass "number" (as a string) or if we pass just number (as an int) does it auto string convert for us ?

livvy10 commented 1 week ago

change uwaid to email which should be unique (for User)