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

Pass stock location to inventory unit factory #3375

Merged
merged 1 commit into from
Oct 30, 2019
Merged

Pass stock location to inventory unit factory #3375

merged 1 commit into from
Oct 30, 2019

Conversation

pelargir
Copy link
Contributor

@pelargir pelargir commented Oct 10, 2019

Description
When building fulfilled stock packages in specs using the stock package factory, this passes the given transient stock location to the inventory units that get built. This prevents the inventory unit factory from building unique stock locations for each inventory unit that gets built, and ensures that the stock package is using the same stock location as the inventory units it contains.

Checklist:

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have updated Guides and README accordingly to this change (if needed)
  • I have added tests to cover this change (if needed)

Copy link
Contributor

@ericsaupe ericsaupe left a comment

Choose a reason for hiding this comment

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

👍

@pelargir
Copy link
Contributor Author

Rebased from master.

When creating fulfilled stock packages in specs, this passes the given
transient stock location to the inventory units that get built. This
prevents the inventory unit factory from building unique stock locations
for each inventory unit that gets built, and ensures that the stock
package is using the same stock location as the inventory units it
contains.
@pelargir
Copy link
Contributor Author

The build is now passing. What else needs to happen before this PR can be merged?

Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Thanks

@kennyadsl kennyadsl merged commit 6036639 into solidusio:master Oct 30, 2019
@kennyadsl
Copy link
Member

Thank you!

@pelargir pelargir deleted the improve-inventory-unit-factory branch October 30, 2019 14:28
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