Show action badge in LHN for individual expense reports#88294

Merged

AndrewGable merged 13 commits into

mainfrom

claude-showActionBadgeForExpenseReports

Apr 21, 2026

Commits on Apr 19, 2026

Show action badge (Approve/Pay/Submit) in LHN for individual expense reports

MelvinBot
and
aimane-chnaif
committed
Fix test to use chat report instead of expense report for deleted action filtering

MelvinBot
and
aimane-chnaif
committed
Move badge computation before isWaitingForAssigneeToCompleteAction early return

MelvinBot
and
aimane-chnaif
committed
Extract getBadgeFromIOUReport to deduplicate badge logic

MelvinBot
and
aimane-chnaif
committed
Address review feedback: scope to hasParentAccess, restore comment, add tests

MelvinBot
and
aimane-chnaif
committed
Refactor isWaitingForAssigneeToCompleteAction to getActionTypeForAssigneeToComplete

MelvinBot
and
aimane-chnaif
committed
Optimize: defer getIOUReportActionWithBadge call until needed

MelvinBot
and
aimane-chnaif
committed
Add unit test for actionBadge on expense reports with hasParentAccess=false

MelvinBot
and
aimane-chnaif
committed
Merge remote-tracking branch 'origin/main' into claude-showActionBadgeForExpenseReports
${"message":"Not Found","documentation_url":"https://docs.github.com/rest/issues/comments#get-an-issue-comment","status":"404"}$
MelvinBot
and
{"message":"Not Found","documentation_url":"https://docs.github.com/rest/issues/comments#get-an-issue-comment","status":"404"}
committed
Fix prettier formatting in ReportUtils.ts
${"message":"Not Found","documentation_url":"https://docs.github.com/rest/issues/comments#get-an-issue-comment","status":"404"}$
MelvinBot
and
{"message":"Not Found","documentation_url":"https://docs.github.com/rest/issues/comments#get-an-issue-comment","status":"404"}
committed
Use getBadgeFromIOUReport directly for expense reports in early return path

MelvinBot
and
aimane-chnaif
committed

Commits on Apr 21, 2026

Merge remote-tracking branch 'origin/main' into claude-showActionBadgeForExpenseReports
MelvinBot
committed
Apply task filtering from #87166 to assignee-complete early return
${"message":"Not Found","documentation_url":"https://docs.github.com/rest/issues/comments#get-an-issue-comment","status":"404"}$
MelvinBot
and
{"message":"Not Found","documentation_url":"https://docs.github.com/rest/issues/comments#get-an-issue-comment","status":"404"}
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Show action badge in LHN for individual expense reports#88294

Show action badge in LHN for individual expense reports#88294
AndrewGable merged 13 commits into
mainfrom
claude-showActionBadgeForExpenseReports

Commits on Apr 19, 2026

Show action badge (Approve/Pay/Submit) in LHN for individual expense reports

Fix test to use chat report instead of expense report for deleted action filtering

Move badge computation before isWaitingForAssigneeToCompleteAction early return

Extract getBadgeFromIOUReport to deduplicate badge logic

Address review feedback: scope to hasParentAccess, restore comment, add tests

Refactor isWaitingForAssigneeToCompleteAction to getActionTypeForAssigneeToComplete

Optimize: defer getIOUReportActionWithBadge call until needed

Add unit test for actionBadge on expense reports with hasParentAccess=false

Merge remote-tracking branch 'origin/main' into claude-showActionBadgeForExpenseReports

Fix prettier formatting in ReportUtils.ts

Use getBadgeFromIOUReport directly for expense reports in early return path

Commits on Apr 21, 2026

Merge remote-tracking branch 'origin/main' into claude-showActionBadgeForExpenseReports

Apply task filtering from #87166 to assignee-complete early return