Open AnsonH opened 2 years ago
This will not be done within the scope of this FYP since Kris is interested in revamping the entire ZINC UI. This issue would remain open for future maintainers to be aware that they can merge two separate pages into a single one.
Description
Currently the YAML config editor page is present into two pages:
/assignments/{assignmentId}/configs/new
/assignments/{assignmentId}/configs/{assignmentConfigId}/yaml
Both pages are very similar except for a few additional features in the "Edit existing assignment" page. Instead of splitting up the code for the two pages into two separate files, we should merge the code into a single file. This avoids code duplication.
Solutions
/assignments/{assignmentId}/configs/{assignmentId}/yaml
, whereassignmentConfigId
has a type ofnumber | "new"
Screenshots
/assignments/{assignmentId}/configs/new
/assignments/{assignmentId}/configs/{assignmentConfigId}/yaml