-
Notifications
You must be signed in to change notification settings - Fork 18
Remove unused frontend code #1609
Copy link
Copy link
Labels
bugSomething isn't workingSomething isn't workingfrontendThis affects the frontend applicationThis affects the frontend applicationmaintenanceA task or required work to maintain the applicationA task or required work to maintain the application
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingfrontendThis affects the frontend applicationThis affects the frontend applicationmaintenanceA task or required work to maintain the applicationA task or required work to maintain the application
Type
Fields
Give feedbackPriority
None yet
Describe the bug
Throughout the years and several redesigns, the frontend codebase has a few functions, Vue-provide statements or exports, that are defined but not used anymore and should be removed.
Findings
The following (wip) list contains potentially removable code snippets.
Note
The lines listed are only the definition. The actual number of lines that can be deleted is bigger in most cases.
appointment/frontend/src/utils.ts
Line 104 in 6965d85
appointment/frontend/src/keys.ts
Line 22 in 6965d85
appointment/frontend/src/keys.ts
Line 26 in 6965d85
appointment/frontend/src/definitions.ts
Line 39 in 6965d85
appointment/frontend/src/definitions.ts
Line 84 in 6965d85
appointment/frontend/src/definitions.ts
Line 94 in 6965d85
appointment/frontend/src/definitions.ts
Line 105 in 6965d85
appointment/frontend/src/definitions.ts
Line 118 in 6965d85
appointment/frontend/src/definitions.ts
Line 182 in 6965d85
appointment/frontend/src/definitions.ts
Line 250 in 6965d85
appointment/frontend/src/models.ts
Line 377 in 6965d85
appointment/frontend/src/models.ts
Line 388 in 6965d85
Additional context
I found those when implementing another feature. So we can just keep an eye open for dead code and throw possible findings in here until we purge the list.