fix: Populate card details as part of handleDragEnd

rcdexta · rcdexta · commit 87f4a8562e99 · 2018-06-30T16:34:55.000-07:00
#97
diff --git a/README.md b/README.md
@@ -90,7 +90,7 @@ This is the container component that encapsulates the lanes and cards
 | collapsibleLanes    | boolean  | Make the lanes with cards collapsible. Default: false                                                                          |
 | editable            | boolean  | Makes the entire board editable. Allow cards to be added or deleted Default: false                                             |
 | handleDragStart     | function | Callback function triggered when card drag is started: `handleDragStart(cardId, laneId)`                                       |
-| handleDragEnd       | function | Callback function triggered when card drag ends: `handleDragEnd(cardId, sourceLaneId, targetLaneId, position)`                 |
+| handleDragEnd       | function | Callback function triggered when card drag ends: `handleDragEnd(cardId, sourceLaneId, targetLaneId, position, cardDetails)`                 |
 | handleLaneDragStart | function | Callback function triggered when lane drag is started: `handleLaneDragStart(laneId)`                                           |
 | handleLaneDragEnd   | function | Callback function triggered when lane drag ends: `handleLaneDragEnd(laneId, newPosition)`                                      |
 | cardDragClass       | string   | CSS class to be applied to Card when being dragged                                                                             |          
diff --git a/src/components/Lane.js b/src/components/Lane.js
@@ -138,7 +138,7 @@ class Lane extends Component {
         cardId: payload.id,
         index: addedIndex
       })
-      handleDragEnd && handleDragEnd(payload.id, payload.laneId, laneId, addedIndex)
+      handleDragEnd && handleDragEnd(payload.id, payload.laneId, laneId, addedIndex, payload)
     }
   }
 
diff --git a/stories/DragDrop.story.js b/stories/DragDrop.story.js
@@ -17,12 +17,14 @@ storiesOf('Drag-n-Drop', module)
         console.log(`laneId: ${laneId}`)
       }
 
-      const handleDragEnd = (cardId, sourceLaneId, targetLaneId, position) => {
+      const handleDragEnd = (cardId, sourceLaneId, targetLaneId, position, card) => {
         console.log('drag ended')
         console.log(`cardId: ${cardId}`)
         console.log(`sourceLaneId: ${sourceLaneId}`)
         console.log(`targetLaneId: ${targetLaneId}`)
         console.log(`newPosition: ${position}`)
+        console.log(`cardDetails:`)
+        console.log(card)
       }
 
       const handleLaneDragStart = laneId => {

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ class Lane extends Component {`
`138`	`138`	`cardId: payload.id,`
`139`	`139`	`index: addedIndex`
`140`	`140`	`})`
`141`		`- handleDragEnd && handleDragEnd(payload.id, payload.laneId, laneId, addedIndex)`
	`141`	`+ handleDragEnd && handleDragEnd(payload.id, payload.laneId, laneId, addedIndex, payload)`
`142`	`142`	`}`
`143`	`143`	`}`
`144`	`144`
Original file line number	Diff line number	Diff line change
`@@ -17,12 +17,14 @@ storiesOf('Drag-n-Drop', module)`
`17`	`17`	console.log(`laneId: ${laneId}`)
`18`	`18`	`}`
`19`	`19`
`20`		`- const handleDragEnd = (cardId, sourceLaneId, targetLaneId, position) => {`
	`20`	`+ const handleDragEnd = (cardId, sourceLaneId, targetLaneId, position, card) => {`
`21`	`21`	`console.log('drag ended')`
`22`	`22`	console.log(`cardId: ${cardId}`)
`23`	`23`	console.log(`sourceLaneId: ${sourceLaneId}`)
`24`	`24`	console.log(`targetLaneId: ${targetLaneId}`)
`25`	`25`	console.log(`newPosition: ${position}`)
	`26`	+ console.log(`cardDetails:`)
	`27`	`+ console.log(card)`
`26`	`28`	`}`
`27`	`29`
`28`	`30`	`const handleLaneDragStart = laneId => {`