Skip to content

feat: remove dead code unit-task-list replaced by f-unit-task-list#484

Open
Sujay-Deakin wants to merge 1 commit intothoth-tech:10.0.xfrom
Sujay-Deakin:investigate/f-unit-task-list-v2
Open

feat: remove dead code unit-task-list replaced by f-unit-task-list#484
Sujay-Deakin wants to merge 1 commit intothoth-tech:10.0.xfrom
Sujay-Deakin:investigate/f-unit-task-list-v2

Conversation

@Sujay-Deakin
Copy link
Copy Markdown

@Sujay-Deakin Sujay-Deakin commented May 3, 2026

Note: This is a cleaned-up version of PR #456, containing only the
relevant files under src/app/units/states/tasks/

Summary

This PR investigates the migration status of unit-task-list and verifies whether f-unit-task-list has fully replaced it, and removes the dead code.

What was checked

  • Searched for all references to unit-task-list
  • Verified usage of f-unit-task-list
  • Compared old and new component responsibilities
  • Tested the viewer flow locally

Findings

  • unit-task-list is no longer actively used in the application
  • The only occurrences are within its own directory and a commented reference in directives.coffee
  • No templates or modules reference the old directive
  • f-unit-task-list is actively used in tasks-viewer.component.html and renders correctly
image

Conclusion

f-unit-task-list has completely replaced unit-task-list. The old AngularJS files have been removed in this PR.

Suggested next step

  • Proceed with migration of viewer.coffee as the next step in the tasks module migration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant