johba/harb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: Misleading 'Tax Paid' label — value includes unpaid tax due (#374)

Browse source 
taxPaidGes = taxDue + taxPaid, so the displayed value includes both
outstanding tax and historically paid tax. Rename the UI label from
'Tax Paid' to 'Tax Cost' to accurately reflect the combined amount.
Update the matching E2E test selector accordingly.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
openhands 2026-03-06 11:51:45 +00:00
parent 98dac2f67e
commit 18a2d4138e
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
tests/e2e/06-dashboard-pages.spec.ts
View file

@ -247,8 +247,8 @@ test.describe('Dashboard Pages', () => {
const currentValue = page.locator('text=/Current Value/i').first(); const currentValue = page.locator('text=/Current Value/i').first();
await expect(currentValue).toBeVisible({ timeout: 5_000 }); await expect(currentValue).toBeVisible({ timeout: 5_000 });
// Should show tax paid // Should show tax cost
const taxPaid = page.locator('text=/Tax Paid/i').first(); const taxPaid = page.locator('text=/Tax Cost/i').first();
await expect(taxPaid).toBeVisible({ timeout: 5_000 }); await expect(taxPaid).toBeVisible({ timeout: 5_000 });
// Should show net return // Should show net return

2
web-app/src/components/collapse/CollapseActive.vue
View file

@ -41,7 +41,7 @@
</div> </div>
<div v-if="!error || taxPaidGes !== undefined" class="profit-stats-wrapper"> <div v-if="!error || taxPaidGes !== undefined" class="profit-stats-wrapper">
<div class="profit-stats-item"> <div class="profit-stats-item">
<div><b>Tax Paid</b></div> <div><b>Tax Cost</b></div>
<div>{{ taxPaidGes !== undefined ? formatTokenAmount(taxPaidGes) : '...' }} $KRK</div> <div>{{ taxPaidGes !== undefined ? formatTokenAmount(taxPaidGes) : '...' }} $KRK</div>
</div> </div>
<div class="profit-stats-item"> <div class="profit-stats-item">