Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delivery to target warehouse #4040

Merged
merged 6 commits into from
Oct 14, 2015
Merged

Conversation

nabinhait
Copy link
Member

Against #3970

"warehouse": "_Test Warehouse - _TC",
"delivery_document_no": "",
"purchase_document_no": se.name
})

def test_delivery_of_bundled_items_to_target_warehouse(self):
Copy link
Contributor

Choose a reason for hiding this comment

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

In this test, we have passed the target warehouse as _Test Warehouse 1 - _TC, but we haven't changed the target warehouse in packed item table to something different. Isn't that also a requirement of #3970 ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Target warehouse get copied from main item table, if not specified in packing list

@anandpdoshi
Copy link
Contributor

@nabinhait test case is failing

@rmehta
Copy link
Member

rmehta commented Sep 28, 2015

Add this:

  • Ability to change Serial No Item in Manufacture / Repack type of operation

@rmehta
Copy link
Member

rmehta commented Oct 13, 2015

@sliska This is due in next release. Sorry for the delay (we were tied up with conference)

@sliska
Copy link

sliska commented Oct 13, 2015

Hi Rushabh,

When will you have time for the second deliverable? (Buy Product Bundles
(#3969))

We had agreed to push it back until October. Will it be ready for the next
release?

thanks,
Sofia

On Tue, Oct 13, 2015 at 2:59 AM, Rushabh Mehta notifications@github.com
wrote:

@sliska https://github.com/sliska This is due in next release. Sorry
for the delay (we were tied up with conference)


Reply to this email directly or view it on GitHub
#4040 (comment).

rmehta added a commit that referenced this pull request Oct 14, 2015
@rmehta rmehta merged commit c27748b into frappe:develop Oct 14, 2015
@nabinhait nabinhait deleted the dn_target_warehouse branch November 19, 2015 09:17
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