feat: Add comments to clarify the need for APPLICATIONSTATUS and SHIFTSTATUS enums in ShiftsRepositoryImpl
This commit is contained in:
@@ -15,6 +15,8 @@ class ShiftsRepositoryImpl
|
|||||||
// Cache: ApplicationID -> RoleID (For Accept/Decline w/ Update mutation)
|
// Cache: ApplicationID -> RoleID (For Accept/Decline w/ Update mutation)
|
||||||
final Map<String, String> _appToRoleIdMap = {};
|
final Map<String, String> _appToRoleIdMap = {};
|
||||||
|
|
||||||
|
// This need to be an APPLICATION
|
||||||
|
// THERE SHOULD BE APPLICATIONSTATUS and SHIFTSTATUS enums in the domain layer to avoid this string mapping and potential bugs.
|
||||||
@override
|
@override
|
||||||
Future<List<Shift>> getMyShifts({
|
Future<List<Shift>> getMyShifts({
|
||||||
required DateTime start,
|
required DateTime start,
|
||||||
|
|||||||
Reference in New Issue
Block a user