Skip to content

TASK-2025-01184:Edited Asset Transfer Request doctype#1033

Merged
MhmdSinanKT merged 2 commits intodevelopfrom
TASK-2025-01184
Jun 10, 2025
Merged

TASK-2025-01184:Edited Asset Transfer Request doctype#1033
MhmdSinanKT merged 2 commits intodevelopfrom
TASK-2025-01184

Conversation

@NaseeraVP
Copy link
Collaborator

Feature description

Need to:

  • Remove the Asset Return Checklist Template field in asset transfer request doctype.
  • Show Asset Return Checklist table with asset or asset in the bundle if it is bundle when workflow state is transfered and received.
  • Disable "Add Row" in Asset Return Checklist Table.
  • Make the checklist item field in Asset Return Checklist table is read-only.

Solution description

  • Removed the Asset Return Checklist Template field in asset transfer request doctype.
  • Showed Asset Return Checklist table with asset or asset in the bundle if it is bundle when workflow state is transfered and received.
  • Disabled "Add Row" in Asset Return Checklist Table.
  • Made the checklist item field in Asset Return Checklist table is read-only.

Output screenshots (optional)

Screencast from 10-06-25 11:37:18 AM IST.webm
Screencast from 10-06-25 11:39:11 AM IST.webm
Screencast from 10-06-25 11:40:41 AM IST.webm

Areas affected and ensured

Asset Transfer Request Doctype

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

No.

Was this feature tested on the browsers?

  • Mozilla Firefox

@NaseeraVP NaseeraVP requested a review from MhmdSinanKT June 10, 2025 06:12
Comment on lines 126 to 127


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unnecessary line breaks

@MhmdSinanKT MhmdSinanKT merged commit c3b2639 into develop Jun 10, 2025
@MhmdSinanKT MhmdSinanKT deleted the TASK-2025-01184 branch June 10, 2025 08:58
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