Skip to content

TASK-2026-00406: Processed only new trip sheet rows in Monthly Consolidated Trip Sheet and prevent duplicate journal entry creation#1459

Merged
MhmdSinanKT merged 1 commit intodevelopfrom
TASK-2026-00406
Mar 26, 2026
Merged

TASK-2026-00406: Processed only new trip sheet rows in Monthly Consolidated Trip Sheet and prevent duplicate journal entry creation#1459
MhmdSinanKT merged 1 commit intodevelopfrom
TASK-2026-00406

Conversation

@NaseeraVP
Copy link
Copy Markdown
Collaborator

@NaseeraVP NaseeraVP commented Mar 26, 2026

Feature description

Need to: Process only new trip sheet rows in Monthly Consolidated Trip Sheet and prevent duplicate journal entry creation

Solution description

  • Add fields:

    is_processed (Check, default = 0)
    processed_in_jv (Link to Journal Entry) in monthly_consolidated_trip_sheet_details child table

  • a flag (is_processed) to indicate whether it has already been included in a JV
    a reference (processed_in_jv) to store the Journal Entry in which it was processed

  • When creating a Journal Entry:

only rows marked as unprocessed are selected
totals are calculated based only on these rows
after successful JV creation, those rows are marked as processed and linked to the created JV

  • During trip sheet fetching:

already processed trip sheets are excluded
trip sheets already added in the current document are also excluded

Output screenshots (optional)

Screencast.from.26-03-26.01.05.09.PM.IST.webm
Screencast.from.26-03-26.01.06.28.PM.IST.webm

Areas affected and ensured

monthly consolidated trip sheet doctype

Is there any existing behaviour change of other features due to this code change?

No.

Was this feature tested on these browsers?

  • Chrome

…Sheet and prevent duplicate jounal entry creation
@NaseeraVP NaseeraVP requested a review from MhmdSinanKT March 26, 2026 08:22
@MhmdSinanKT MhmdSinanKT merged commit ed86a48 into develop Mar 26, 2026
@MhmdSinanKT MhmdSinanKT deleted the TASK-2026-00406 branch March 26, 2026 09:11
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