Open
Description
While implementing the new UI for bug reporting phase in #1312, I noticed some places that I could be further improved in terms of code quality, mainly the following 2 aspects:
- Duplication of code between
deleteIssue
andundeleteIssue
, they are essentially performing the same actions, but in reverse directions, hence the code is highly duplicated. - There are 2 separate lists
issuesPendingDeletion
andissuesPendingRestore
, which is used for displaying the loading icon (instead of the delete/restore icon), we can actually merge these 2 lists into a single list (maybeissuesPendingAction
?) instead and display the loading icon (and hide the delete/restore icon) based on this list - There is a lot of conditional logic in the
issue-tables.component.html
file, we should probably move these logic into theissue-tables.component.ts
file instead. See the*ngIf=...
