feat: updates for cashier, tds reports, rbi reserve and dashboard
This commit is contained in:
@@ -116,8 +116,8 @@
|
||||
<div class="f121__note f121__note--warn">
|
||||
<mat-icon>warning_amber</mat-icon>
|
||||
<div>
|
||||
A Form 121 declaration is on record for this year, but withholding is still switched on for one or more
|
||||
deposits. Turn it off on those accounts, or tax will keep being deducted.
|
||||
A declaration is on record for this year, but one or more deposits still carry a tax group. Those accounts
|
||||
deduct on their own and ignore the declaration — remove the tax group from them.
|
||||
</div>
|
||||
</div>
|
||||
}
|
||||
@@ -192,20 +192,90 @@
|
||||
}
|
||||
</span>
|
||||
</div>
|
||||
@if (declarationOnRecord && declarationFy) {
|
||||
@if (declarationRecord; as d) {
|
||||
<div class="f121__kv-row">
|
||||
<span>Declared for</span>
|
||||
<span>FY {{ declarationFy }}</span>
|
||||
<span>FY {{ d.fy }} · Form {{ d.formType }}</span>
|
||||
</div>
|
||||
}
|
||||
@if (declarationOnRecord && !acceptanceDateKnown) {
|
||||
<div class="f121__kv-row f121__kv-row--muted">
|
||||
<span>Acceptance date</span>
|
||||
<div class="f121__kv-row">
|
||||
<span>Accepted on</span>
|
||||
<span>
|
||||
Not captured — the declaration is treated as covering the whole year. A mid-year acceptance would change
|
||||
the figures below.
|
||||
{{ d.acceptedOn | date: 'd MMM y' }} — no TDS on interest credited from this day to 31 March. Interest
|
||||
credited before it is taxed as usual.
|
||||
</span>
|
||||
</div>
|
||||
@if (d.uin) {
|
||||
<div class="f121__kv-row">
|
||||
<span>UIN</span>
|
||||
<span class="f121__mono">{{ d.uin }}</span>
|
||||
</div>
|
||||
}
|
||||
<div class="f121__kv-row">
|
||||
<span></span>
|
||||
<span>
|
||||
<button type="button" class="f121__btn f121__btn--warn" (click)="revokeDeclaration()">
|
||||
Withdraw declaration
|
||||
</button>
|
||||
</span>
|
||||
</div>
|
||||
} @else if (fy && !configMissing) {
|
||||
<div class="f121__kv-row">
|
||||
<span></span>
|
||||
<span>
|
||||
@if (!showDeclarationForm) {
|
||||
<button type="button" class="f121__btn" (click)="openDeclarationForm()" [disabled]="!panWellFormed">
|
||||
Record {{ formLabel }} for FY {{ fy }}
|
||||
</button>
|
||||
@if (!panWellFormed) {
|
||||
<div class="muted">A valid PAN must be on the KYC first — a declaration without one is void.</div>
|
||||
}
|
||||
}
|
||||
</span>
|
||||
</div>
|
||||
@if (showDeclarationForm) {
|
||||
<div class="f121__decl-form">
|
||||
<label>
|
||||
<span>Accepted on *</span>
|
||||
<input type="date" [(ngModel)]="newDeclaration.acceptedOn" />
|
||||
</label>
|
||||
<label>
|
||||
<span>UIN</span>
|
||||
<input type="text" [(ngModel)]="newDeclaration.uin" placeholder="Allotted on acceptance" />
|
||||
</label>
|
||||
<label>
|
||||
<span>Estimated income for the year (₹)</span>
|
||||
<input type="number" min="0" [(ngModel)]="newDeclaration.estimatedIncome" />
|
||||
</label>
|
||||
@if (declarationError) {
|
||||
<div class="f121__note f121__note--warn">
|
||||
<mat-icon>error_outline</mat-icon>
|
||||
<div>{{ declarationError }}</div>
|
||||
</div>
|
||||
}
|
||||
<div class="f121__decl-actions">
|
||||
<button type="button" class="f121__btn" (click)="showDeclarationForm = false">Cancel</button>
|
||||
<button
|
||||
type="button"
|
||||
class="f121__btn f121__btn--primary"
|
||||
(click)="saveDeclaration()"
|
||||
[disabled]="savingDeclaration"
|
||||
>
|
||||
{{ savingDeclaration ? 'Saving…' : 'Save declaration' }}
|
||||
</button>
|
||||
</div>
|
||||
</div>
|
||||
}
|
||||
}
|
||||
@for (old of declarations; track old.id) {
|
||||
@if (old.revoked && old.fy === fy) {
|
||||
<div class="f121__kv-row f121__kv-row--muted">
|
||||
<span>Withdrawn</span>
|
||||
<span>
|
||||
Form {{ old.formType }} accepted {{ old.acceptedOn | date: 'd MMM y' }}, withdrawn
|
||||
{{ old.revokedOn | date: 'd MMM y' }} — {{ old.revokeReason }}
|
||||
</span>
|
||||
</div>
|
||||
}
|
||||
}
|
||||
@if (eligibility) {
|
||||
@for (reason of eligibility.reasons; track reason) {
|
||||
|
||||
@@ -332,3 +332,61 @@
|
||||
font-family: ui-monospace, SFMono-Regular, Menlo, Consolas, monospace;
|
||||
font-weight: 600;
|
||||
}
|
||||
|
||||
/* ── Recording a declaration ── */
|
||||
.f121__btn {
|
||||
padding: 6px 14px;
|
||||
border: 1px solid #d1d5db;
|
||||
border-radius: 6px;
|
||||
background: #fff;
|
||||
font: inherit;
|
||||
font-size: 12.5px;
|
||||
font-weight: 600;
|
||||
cursor: pointer;
|
||||
|
||||
&:disabled {
|
||||
opacity: 0.55;
|
||||
cursor: not-allowed;
|
||||
}
|
||||
|
||||
&--primary {
|
||||
border-color: #1d4ed8;
|
||||
background: #1d4ed8;
|
||||
color: #fff;
|
||||
}
|
||||
|
||||
&--warn {
|
||||
border-color: #b45309;
|
||||
color: #b45309;
|
||||
}
|
||||
}
|
||||
|
||||
.f121__decl-form {
|
||||
display: grid;
|
||||
gap: 10px;
|
||||
padding: 12px 16px;
|
||||
|
||||
label {
|
||||
display: grid;
|
||||
gap: 4px;
|
||||
font-size: 12px;
|
||||
|
||||
span {
|
||||
color: #6b7280;
|
||||
font-weight: 600;
|
||||
}
|
||||
|
||||
input {
|
||||
max-width: 280px;
|
||||
padding: 6px 8px;
|
||||
border: 1px solid #d1d5db;
|
||||
border-radius: 6px;
|
||||
font: inherit;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
.f121__decl-actions {
|
||||
display: flex;
|
||||
gap: 8px;
|
||||
}
|
||||
|
||||
@@ -23,6 +23,8 @@ import { SavingsService } from 'app/savings/savings.service';
|
||||
import { FixedDepositsService } from 'app/deposits/fixed-deposits/fixed-deposits.service';
|
||||
import { RecurringDepositsService } from 'app/deposits/recurring-deposits/recurring-deposits.service';
|
||||
import { STANDALONE_SHARED_IMPORTS } from 'app/standalone-shared.module';
|
||||
import { TdsDeclarationRecord, TdsReportsService } from 'app/tds-reports/tds-reports.service';
|
||||
import { TdsDeductee } from 'app/tds-reports/tds-reports-data';
|
||||
|
||||
/** Domain logic */
|
||||
import {
|
||||
@@ -111,6 +113,7 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
private savingsService = inject(SavingsService);
|
||||
private fixedDepositsService = inject(FixedDepositsService);
|
||||
private recurringDepositsService = inject(RecurringDepositsService);
|
||||
private tdsService = inject(TdsReportsService);
|
||||
|
||||
private destroy$ = new Subject<void>();
|
||||
|
||||
@@ -124,23 +127,25 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
pan = '';
|
||||
panWellFormed = false;
|
||||
|
||||
// ── Declaration on record ──
|
||||
declarationOnRecord = false;
|
||||
// ── Declarations on record ──
|
||||
/**
|
||||
* Every declaration the customer has given, each for one year and from the day the branch
|
||||
* accepted it. The KYC record only ever held a type and a year, which made a September
|
||||
* declaration shield interest back to April and let last year's form run on.
|
||||
*/
|
||||
declarations: TdsDeclarationRecord[] = [];
|
||||
/**
|
||||
* The form actually in force for the selected year. From FY 2026-27 a single
|
||||
* Form 121 replaced the 15G/15H split, so this is derived from the year rather
|
||||
* than from the payee's age.
|
||||
*/
|
||||
declarationForm: DeclarationForm = '121';
|
||||
/** Kind recorded at KYC, meaningful only for a pre-2026 year. */
|
||||
declarationKind: '15G' | '15H' = '15G';
|
||||
declarationFy = '';
|
||||
/**
|
||||
* The KYC step records only the year, not the date the branch accepted the
|
||||
* declaration, so it has to be treated as covering the whole year. Surfaced on
|
||||
* screen rather than hidden — a mid-year acceptance would change the figures.
|
||||
*/
|
||||
acceptanceDateKnown = false;
|
||||
|
||||
// ── Recording a declaration ──
|
||||
showDeclarationForm = false;
|
||||
savingDeclaration = false;
|
||||
declarationError = '';
|
||||
newDeclaration = { acceptedOn: '', uin: '', estimatedIncome: null as number | null };
|
||||
|
||||
eligibility: EligibilityResult | null = null;
|
||||
|
||||
@@ -157,9 +162,9 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
configMissing = false;
|
||||
|
||||
/** Raw account inputs, kept so the year selector can recompute cheaply. */
|
||||
private accountInputs: (TdsAccountInput & { meta: Form121AccountRow })[] = [];
|
||||
/** Withheld-tax postings by FY, from the ledger. */
|
||||
private actualTdsByFy = new Map<string, number>();
|
||||
private accountInputs: (TdsAccountInput & { meta: Form121AccountRow; taxLines: InterestCredit[] })[] = [];
|
||||
/** This customer's FD/RD interest received in savings, and TDS taken there, per year. */
|
||||
private platformByFy = new Map<string, TdsDeductee | null>();
|
||||
|
||||
ngOnInit(): void {
|
||||
this.route.parent.params.pipe(takeUntil(this.destroy$)).subscribe((params: any) => {
|
||||
@@ -182,13 +187,15 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
forkJoin({
|
||||
client: this.clientsService.getClientData(this.clientId, true).pipe(catchError(() => of(null))),
|
||||
kyc: this.clientsService.getClientKyc(this.clientId, true).pipe(catchError(() => of(null))),
|
||||
accounts: this.clientsService.getClientAccountData(this.clientId).pipe(catchError(() => of(null)))
|
||||
accounts: this.clientsService.getClientAccountData(this.clientId).pipe(catchError(() => of(null))),
|
||||
declarations: this.tdsService.getDeclarations(Number(this.clientId)).pipe(catchError(() => of([])))
|
||||
})
|
||||
.pipe(
|
||||
takeUntil(this.destroy$),
|
||||
switchMap(({ client, kyc, accounts }) => {
|
||||
switchMap(({ client, kyc, accounts, declarations }) => {
|
||||
this.applyClient(client);
|
||||
this.applyKyc(kyc);
|
||||
this.declarations = declarations || [];
|
||||
const deposits = this.depositAccountsOf(accounts);
|
||||
return deposits.length ? forkJoin(deposits.map((a) => this.fetchAccount(a))) : of([] as any[]);
|
||||
}),
|
||||
@@ -212,12 +219,8 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
private applyKyc(kyc: any): void {
|
||||
this.pan = (kyc?.panNumber ?? '').toUpperCase();
|
||||
this.panWellFormed = isWellFormedPan(this.pan);
|
||||
this.declarationOnRecord = !!kyc?.form15Applicable;
|
||||
this.declarationKind = kyc?.form15Type === '15H' ? '15H' : '15G';
|
||||
this.declarationForm =
|
||||
this.fy && FY_TAX_CONFIG[normaliseFy(this.fy)] ? declarationFormFor(this.fy, this.dateOfBirth) : '121';
|
||||
this.declarationFy = kyc?.form15FY ?? '';
|
||||
this.acceptanceDateKnown = false;
|
||||
}
|
||||
|
||||
/** Savings, FD and RD accounts — everything that can credit interest. */
|
||||
@@ -251,7 +254,7 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
*/
|
||||
private ingest(details: any[]): void {
|
||||
this.accountInputs = [];
|
||||
this.actualTdsByFy = new Map<string, number>();
|
||||
this.platformByFy = new Map<string, TdsDeductee | null>();
|
||||
const fys = new Set<string>();
|
||||
const clientIdNum = Number(this.clientId);
|
||||
|
||||
@@ -272,6 +275,7 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
};
|
||||
|
||||
const credits: InterestCredit[] = [];
|
||||
const taxLines: InterestCredit[] = [];
|
||||
let tdsDeducted = 0;
|
||||
|
||||
for (const t of d.transactions ?? []) {
|
||||
@@ -289,8 +293,7 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
} else if (t.transactionType?.withholdTax) {
|
||||
const amount = t.amount ?? 0;
|
||||
tdsDeducted += amount;
|
||||
const txnFy = fyOf(iso);
|
||||
this.actualTdsByFy.set(txnFy, (this.actualTdsByFy.get(txnFy) ?? 0) + amount);
|
||||
taxLines.push({ date: iso, amount, accountNo: ref.accountNo });
|
||||
}
|
||||
}
|
||||
|
||||
@@ -313,12 +316,12 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
deducteeClientId: deductee
|
||||
};
|
||||
|
||||
this.accountInputs.push({ ...ref, credits, meta });
|
||||
this.accountInputs.push({ ...ref, credits, taxLines, meta });
|
||||
}
|
||||
|
||||
this.availableFys = [...fys].sort().reverse();
|
||||
if (!this.availableFys.includes(this.fy)) {
|
||||
this.fy = this.availableFys[0] ?? (this.declarationFy ? normaliseFy(this.declarationFy) : '');
|
||||
this.fy = this.availableFys[0] ?? this.declarations[0]?.fy ?? Object.keys(FY_TAX_CONFIG).sort().reverse()[0] ?? '';
|
||||
}
|
||||
}
|
||||
|
||||
@@ -338,11 +341,35 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
const year = normaliseFy(this.fy);
|
||||
this.configMissing = !FY_TAX_CONFIG[year];
|
||||
|
||||
// TDS is tracked on FD/RD interest once it reaches savings, and deducted there. Those figures live
|
||||
// on the platform, not in the deposit's own transactions, so they are fetched once per year and
|
||||
// the computation waits for them.
|
||||
if (!this.platformByFy.has(year)) {
|
||||
this.loadPlatformYear(year);
|
||||
return;
|
||||
}
|
||||
const platform = this.platformByFy.get(year);
|
||||
for (const a of this.accountInputs) {
|
||||
if (!a.meta.tdsApplicable) {
|
||||
continue;
|
||||
}
|
||||
const match = platform?.accounts.find((p) => p.accountNo === a.accountNo);
|
||||
a.credits = match?.credits ?? [];
|
||||
a.taxLines = match?.deductions ?? [];
|
||||
}
|
||||
|
||||
// Per-account interest for the selected year, for the three lists.
|
||||
for (const a of this.accountInputs) {
|
||||
a.meta.interest = a.credits.filter((c) => fyOf(c.date) === year).reduce((s, c) => s + c.amount, 0);
|
||||
// The selected year only - this used to total every year the account had ever been taxed in.
|
||||
a.meta.tdsDeducted = a.meta.tdsApplicable
|
||||
? a.taxLines.filter((c) => fyOf(c.date) === year).reduce((s, c) => s + c.amount, 0)
|
||||
: 0;
|
||||
}
|
||||
this.actualTds = this.actualTdsByFy.get(year) ?? 0;
|
||||
// What was actually debited for this customer as deductee, wherever the savings account sits.
|
||||
this.actualTds = this.accountInputs
|
||||
.filter((a) => a.meta.tdsApplicable)
|
||||
.reduce((s, a) => s + a.meta.tdsDeducted, 0);
|
||||
|
||||
if (!this.dateOfBirth || this.configMissing) {
|
||||
return;
|
||||
@@ -382,21 +409,95 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
return applicable.some((a) => a.meta.mode === 'QUARTERLY') ? 'QUARTERLY' : 'ANNUAL';
|
||||
}
|
||||
|
||||
/** The declaration, but only when it was filed for the year being shown. */
|
||||
private declarationForSelectedFy(): Declaration | null {
|
||||
if (!this.declarationOnRecord || !this.fy) {
|
||||
private loadPlatformYear(year: string): void {
|
||||
this.tdsService
|
||||
.getDepositInterest(Number(year.slice(0, 4)), Number(this.clientId))
|
||||
.pipe(
|
||||
catchError(() => of([])),
|
||||
takeUntil(this.destroy$)
|
||||
)
|
||||
.subscribe((rows) => {
|
||||
this.platformByFy.set(year, rows.find((r) => `${r.clientId}` === `${this.clientId}`) ?? null);
|
||||
this.recompute();
|
||||
});
|
||||
}
|
||||
|
||||
/** The live declaration recorded for the year being shown, if any. */
|
||||
get declarationRecord(): TdsDeclarationRecord | null {
|
||||
if (!this.fy) {
|
||||
return null;
|
||||
}
|
||||
const year = normaliseFy(this.fy);
|
||||
if (this.declarationFy && normaliseFy(this.declarationFy) !== year) {
|
||||
return this.declarations.find((d) => d.fy === year && !d.revoked) ?? null;
|
||||
}
|
||||
|
||||
get declarationOnRecord(): boolean {
|
||||
return !!this.declarationRecord;
|
||||
}
|
||||
|
||||
/** The declaration as the engine takes it — operative from the day it was accepted. */
|
||||
private declarationForSelectedFy(): Declaration | null {
|
||||
const d = this.declarationRecord;
|
||||
if (!d) {
|
||||
return null;
|
||||
}
|
||||
return {
|
||||
kind: this.declarationKind,
|
||||
// No acceptance date on record — treated as covering the whole year.
|
||||
acceptedOn: `${year.slice(0, 4)}-04-01`,
|
||||
fy: year
|
||||
};
|
||||
const kind = d.formType === '15H' ? '15H' : d.formType === '15G' ? '15G' : formKindFor(this.dateOfBirth, d.fy);
|
||||
return { kind, acceptedOn: d.acceptedOn, fy: d.fy, uin: d.uin || undefined };
|
||||
}
|
||||
|
||||
openDeclarationForm(): void {
|
||||
this.newDeclaration = { acceptedOn: new Date().toISOString().slice(0, 10), uin: '', estimatedIncome: null };
|
||||
this.declarationError = '';
|
||||
this.showDeclarationForm = true;
|
||||
}
|
||||
|
||||
saveDeclaration(): void {
|
||||
if (!this.newDeclaration.acceptedOn) {
|
||||
this.declarationError = 'Enter the date the branch accepted the form.';
|
||||
return;
|
||||
}
|
||||
this.savingDeclaration = true;
|
||||
this.declarationError = '';
|
||||
this.tdsService
|
||||
.createDeclaration({
|
||||
clientId: Number(this.clientId),
|
||||
fy: normaliseFy(this.fy),
|
||||
formType: declarationFormFor(this.fy, this.dateOfBirth),
|
||||
acceptedOn: this.newDeclaration.acceptedOn,
|
||||
uin: this.newDeclaration.uin || undefined,
|
||||
estimatedIncome: this.newDeclaration.estimatedIncome
|
||||
})
|
||||
.subscribe({
|
||||
next: (rows) => {
|
||||
this.savingDeclaration = false;
|
||||
this.showDeclarationForm = false;
|
||||
this.declarations = rows || [];
|
||||
this.recompute();
|
||||
},
|
||||
error: (err: any) => {
|
||||
this.savingDeclaration = false;
|
||||
this.declarationError =
|
||||
err?.error?.errors?.[0]?.defaultUserMessage || err?.error?.defaultUserMessage || 'Could not save.';
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
revokeDeclaration(): void {
|
||||
const d = this.declarationRecord;
|
||||
if (!d) {
|
||||
return;
|
||||
}
|
||||
const reason = prompt('Why is this declaration being withdrawn? From today, TDS will be deducted again.');
|
||||
if (!reason) {
|
||||
return;
|
||||
}
|
||||
this.tdsService.revokeDeclaration(d.id, reason).subscribe({
|
||||
next: (rows) => {
|
||||
this.declarations = rows || [];
|
||||
this.recompute();
|
||||
},
|
||||
error: (err: any) => alert(err?.error?.defaultUserMessage || 'Could not revoke the declaration.')
|
||||
});
|
||||
}
|
||||
|
||||
// ───────────────────────── Template helpers ─────────────────────────
|
||||
@@ -474,11 +575,12 @@ export class Form121TabComponent implements OnInit, OnDestroy {
|
||||
}
|
||||
|
||||
/**
|
||||
* A declaration is on record but deposit accounts are still set to withhold —
|
||||
* the branch has not switched them off.
|
||||
* A declaration is on record but a deposit still carries a stock Fineract tax group. Those
|
||||
* accounts withhold on their own rules and never consult the declaration record, so the form
|
||||
* does not stop their deduction until the tax group is removed.
|
||||
*/
|
||||
get declarationNotAppliedToAccounts(): boolean {
|
||||
return !!this.declarationForSelectedFy() && this.liableAccounts.some((a) => a.withHoldTax);
|
||||
return !!this.declarationForSelectedFy() && this.liableAccounts.some((a) => !!a.taxGroup);
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
Reference in New Issue
Block a user