Skip to content

Remove priority and category from tasks#64

Merged
JoeProgrammer88 merged 4 commits intomainfrom
copilot/remove-priority-and-category-from-tasks
Mar 26, 2026
Merged

Remove priority and category from tasks#64
JoeProgrammer88 merged 4 commits intomainfrom
copilot/remove-priority-and-category-from-tasks

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 26, 2026

priority and category fields on tasks added complexity without consistent usage. This removes them entirely from the data model, UI, logic, and tests.

Data model (storage.ts)

  • Dropped priority and category from the Task interface
  • Removed priority: 'medium' default in addTask
  • updateCategory / deleteCategory no longer propagate to tasks (habits and finance unaffected)

App logic (app.ts)

  • Removed category filter dropdown logic and group-by (priority/category) from filterTasks
  • Deleted updateCategoryFilter — no longer needed
  • Removed category/priority from task modal (open, save), repeated task creation, and project detail task list
  • Dropped categoryFilter and groupBy from filter persistence (saveFilterSettings / loadFilterSettings / resetFilters)
  • Upcoming tasks now sort by due date then title (replaces priority tiebreaker)

UI (index.html)

  • Removed category filter <select>, group-by <select>, and task form fields for category and priority

Tests (storage.test.ts)

  • Removed priority from addTask calls and assertions
  • Category propagation tests updated to use habits only

E2E tests (e2e/app.spec.ts)

  • Removed the task group by describe block (group by priority, group by category, revert to default grouping, show group by dropdown)
  • Removed should persist groupBy filter across page reloads test
  • Removed groupBySelect interactions and assertions from the reset filters test
  • Removed stale priority fields from task objects in dashboard tests
Original prompt

This section details on the original issue you should resolve

<issue_title>Remove priority and category from tasks</issue_title>
<issue_description>Removing the priority and category fields from tasks will simplify the app. These fields are not being used consistently by users

Associated tests will need to be updated</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Co-authored-by: JoeProgrammer88 <7156063+JoeProgrammer88@users.noreply.github.com>
Agent-Logs-Url: https://github.com/SpeakingInBits/TaskManagerWeb/sessions/12a5e106-fdee-47b6-89c6-cb0b1ab69597
Copilot AI changed the title [WIP] Remove priority and category fields from tasks Remove priority and category from tasks Mar 26, 2026
Copilot AI requested a review from JoeProgrammer88 March 26, 2026 03:14
@JoeProgrammer88 JoeProgrammer88 marked this pull request as ready for review March 26, 2026 03:45
Copy link
Copy Markdown
Member

@JoeProgrammer88 JoeProgrammer88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to remove the e2e tests relating to the task grouping dropdown since the category and priority are being removed

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 26, 2026

We need to remove the e2e tests relating to the task grouping dropdown since the category and priority are being removed

Done in 508bff1. Removed the entire task group by describe block (group by priority, group by category, revert to default grouping tests), the should persist groupBy filter across page reloads test, cleaned up groupBySelect assertions from the reset filters test, and removed stale priority fields from task objects in the dashboard tests.

Copy link
Copy Markdown
Member

@JoeProgrammer88 JoeProgrammer88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests managing tasks must have the task priority removed

Co-authored-by: JoeProgrammer88 <7156063+JoeProgrammer88@users.noreply.github.com>
Agent-Logs-Url: https://github.com/SpeakingInBits/TaskManagerWeb/sessions/23869eb1-ee52-4186-80ec-3623ab3944e7
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 26, 2026

Tests managing tasks must have the task priority removed

Done in 42eb345. Removed the selectOption('#taskPriority', 'high') line from the should create a new task e2e test.

@JoeProgrammer88 JoeProgrammer88 merged commit 9857847 into main Mar 26, 2026
1 check passed
@JoeProgrammer88 JoeProgrammer88 deleted the copilot/remove-priority-and-category-from-tasks branch March 26, 2026 04:39
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.

Remove priority and category from tasks

2 participants