You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There's 2 goals to this ticket: (1) fix a bad-practice coding habit (that greatly increases the chance of errors); and (2) improve the readability of application status labels (e.g., not_registered -> Not Registered)
Acceptance Criteria
remove all manual assignments of status in code with enums (e.g., we don't want to use operation.status = 'Pending' bc it's bad practice. Replace with operation.status = OperationStatus.PENDING)
remove hacky string fixes to status strings in FE (such as calling capitalizeString() in Operations table)
rework the Status labels in Django models. Sepehr's suggestion is to use
Describe the task
There's 2 goals to this ticket: (1) fix a bad-practice coding habit (that greatly increases the chance of errors); and (2) improve the readability of application status labels (e.g.,
not_registered
->Not Registered
)Acceptance Criteria
operation.status = 'Pending'
bc it's bad practice. Replace withoperation.status = OperationStatus.PENDING
)capitalizeString()
in Operations table)Additional context
The text was updated successfully, but these errors were encountered: