Problem/Motivation
Customer Service Representatives (CSRs) experience workflow disruption when modifying order items. The current system requires navigation to the order edit screen for simple tasks like quantity changes or item removal, breaking their flow and slowing down order processing.
Steps to reproduce
- Navigate to an order view page as a CSR
- Attempt to modify an order item quantity or remove an item
- Notice that editing requires navigating to the order edit screen
- Experience workflow interruption and time loss during order processing
Proposed resolution
Add an edit action to each order item on the main order view that opens a focused modal for quantity changes and item removal. This eliminates unnecessary navigation and keeps CSRs in their workflow context.
Remaining tasks
- Add edit button/action to order items on order view
- Implement order item editing modal functionality
- Display essential item details (product name, unit price, quantity, total)
- Add real-time price calculations for quantity changes (TBD)
- Implement "Save Changes" functionality with order refresh
- Add "Cancel" action that discards changes cleanly
- Create "Remove Item" functionality with confirmation dialog
- Implement proper validation for positive numbers only
- Add error handling for network issues and validation failures
- Maintain data integrity during concurrent edits
User interface changes
New: Edit action/button for each order item on order view
New: Modal dialog for editing order items with the following elements:
- Product name display
- Current unit price display
- Quantity input field (positive numbers only)
- Calculated total price with real-time updates
- "Save Changes" button
- "Cancel" button
- "Remove Item" button with confirmation dialog
Issue fork commerce-3545907
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
rinasek commentedComment #5
tbkot commentedComment #6
jsacksick commentedMarking this as needs work as there are conflicts with the MR.
Comment #7
tbkot commentedComment #9
jsacksick commentedMerged, thank your for this Dmytrii.