Skip to content

Conversation

@maniakjaniak
Copy link
Contributor

No description provided.

Copy link

@tabby-or-not tabby-or-not left a comment

Choose a reason for hiding this comment

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

Looks good!

@maniakjaniak maniakjaniak merged commit 2fc0a4e into master May 21, 2018
@maniakjaniak maniakjaniak deleted the maniak/passing-object-to-cells branch May 21, 2018 07:05
@tabby-or-not
Copy link

@jarekwg is this going to have any problem with react-dnd? d4c5025

@jarekwg
Copy link
Member

jarekwg commented May 21, 2018

Potentially.. will have a deeper look tomorrow.

@jarekwg
Copy link
Member

jarekwg commented May 22, 2018

looks like it might be fine, but the work i did there was a bit fiddly, so would recommend double checking that everything still works. (ie lineitems can be rearranged on invoices/quotes/POs)

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.

4 participants