Skip to content

Add "Hide Done Tasks" Checkbox to Task Board#66

Open
Lightning11wins wants to merge 2 commits into
masterfrom
add-hide-done-tasks
Open

Add "Hide Done Tasks" Checkbox to Task Board#66
Lightning11wins wants to merge 2 commits into
masterfrom
add-hide-done-tasks

Conversation

@Lightning11wins

Copy link
Copy Markdown

This PR will add a "hide done tasks" checkbox to the task page of the taskboard.

I hacked together a really janky JS script during Code-a-Thon 2025 that I could paste into my console every time I went to this tab that would do this. After the interns had the same issue, I decided I'd finally fix it. With this PR, there will finally be an official, far-more-user-friendly solution.

@Lightning11wins Lightning11wins self-assigned this Jun 10, 2026
@Lightning11wins Lightning11wins added the size: medium Might be hard to review, usually less than ~5000 lines. label Jun 10, 2026

@gbeeley gbeeley left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

One quick change. Thanks!

Comment thread kardia-app/modules/task/myprojects.cmp Outdated
@Lightning11wins

Copy link
Copy Markdown
Author

@gbeeley I think I've resolved your comments.

@Lightning11wins Lightning11wins requested review from gbeeley and removed request for nboard June 18, 2026 22:23
@Lightning11wins Lightning11wins changed the title Add a checkbox to the task board that hides done tasks. Add "Hide Done Tasks" Checkbox to Task Board Jun 18, 2026

@gbeeley gbeeley left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Odd change to the SQL.

ppl = (select count(1) from /apps/kardia/data/Kardia_DB/t_participant/rows p where :p:t_project_id = :tmp:t_project_id),
spr = (select count(1) from /apps/kardia/data/Kardia_DB/t_sprint_project/rows sp where :sp:t_project_id = :tmp:t_project_id),
past_end = condition(isnull(:pr:t_project_end, getdate()) < getdate() and (select isnull(max(:s:t_sprint_end), getdate()) from /apps/kardia/data/Kardia_DB/t_sprint/rows s, /apps/kardia/data/Kardia_DB/t_sprint_project/rows sp where :sp:t_project_id = :pr:t_project_id and :s:t_sprint_id = :sp:t_sprint_id) < getdate(), 1, 0)
" + (select sum(", tasks_" + :ts:t_task_state_id + " = (select * from expression('/apps/kardia/modules/task/block_tasklist.qy?project=' + :pr:t_project_id + '&state=" + :ts:t_task_state_id + "')) ") from /apps/kardia/data/Kardia_DB/t_task_state/rows ts) + "

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why was this line removed (and the previous similar one)? This provides the itemlist data needed for the task cells.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Oops, I didn't intend to commit that. It's because I run the task board without test data, so those lines prevent the page from rendering. One sec.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yeah, I did not intend to commit that change. Sorry about that. Fixed.

@Lightning11wins

Copy link
Copy Markdown
Author

@gbeeley I think I've resolved your comments.

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

Labels

size: medium Might be hard to review, usually less than ~5000 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants