Skip to content

TASK-2026-00376: Bureau Trip Sheet and Monthly Consolidated Trip Sheet Enhancements for Bata, Fuel Expense, OT, and Supplier Settlement Calculation#1456

Merged
MhmdSinanKT merged 5 commits intodevelopfrom
create_batta_claim_bureau_ts
Mar 18, 2026
Merged

Conversation

@ajmalroshan123
Copy link
Copy Markdown
Contributor

Feature description

Bureau Trip Sheet and Monthly Consolidated Trip Sheet updates: parent-only OT/total hours and batta calculation (no child table), correct total OT batta on save, fuel card expense deduction from bureau fuel card, and settlement via Journal Entry instead of Purchase Invoice

Solution description

Bureau Trip Sheet: calculate_hours_and_days() and calculate_ot_batta() now work on parent fields (starting_date_and_time, ending_date_and_time, total_hours, ot_batta). Server-side calculate_total_ot_batta() computes OT using get_ot_working_hours(supplier) so saved docs show the right total. Renamed settlement button/add vs submit to avoid calling the API on refresh; added supplier/total_hours checks before calling get_ot_working_hours.

Monthly Consolidated Trip Sheet: Server sets total_distance_travelled, total_fuel_consumed, and total_fuel_expense on validate. Entered total_fuel_card_expense is validated against bureau fuel card limit and applied in on_update (and restored on delete). Replaced Purchase Invoice with Journal Entry creation: debits (Batta, OT, Fuel Expense) and credits (Fuel Card, Advance) from Beams Accounts Settings, balancing on Supplier; set party_type/party on Payable/Receivable rows so ERPNext validation passes.

Output screenshots (optional)

Screenshot from 2026-03-17 15-13-10 Screenshot from 2026-03-17 15-11-13 Screenshot from 2026-03-17 15-10-59

Was this feature tested on these browsers?

  • Chrome

@MhmdSinanKT MhmdSinanKT merged commit 685f8ab into develop Mar 18, 2026
@MhmdSinanKT MhmdSinanKT deleted the create_batta_claim_bureau_ts branch March 18, 2026 10:33
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.

2 participants