-
Notifications
You must be signed in to change notification settings - Fork 8
Open
Description
- [PS] refactor relationships for plural/singular relation names
- [PS] update all instances of validation to use new
Validation.pm - use
->insertwherever->new(...)is used for row creation -
addfetch_with_defaultsas an alias fordiscard_changesand employ everywhereadd_to_$relis used - [PS] eliminate dependency on
List::MoreUtils(firstval) - [AP] update all
$selfto use more appropriate naming convention - [GR] switch tests in
t/dbto useTest2::V0 - [AP] Instead of throwing errors on fields not in the DB, filter out non-existent fields. This is done in
ResultSet::UserSet::addUserSet - [PS] DeleteXXXX in ResultSet classes should return
200 OKand the tests should check to make sure that the XXX was deleted. - resolve UserProblem
problem_version(nullable with default value?) add uniqueness on user+problem+version - Attempt field
commentsis a JSON column? - Include a created_date and updated_date field in each table
- Uniformize test checking of boolean fields (and update DB to ensure proper booleans on create/update/fetch)
Metadata
Metadata
Assignees
Labels
No labels