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

feat(stock): rm 0 quantity in decrease adjustment #4097

Conversation

jniles
Copy link
Collaborator

@jniles jniles commented Jan 15, 2020

Adds a feature requested by our clients in Vanga to remove lots with 0 quantity from the grid if you are doing a downwards adjustment.

Closes #4062.

This is what it looks like:

emywtIplwW

Adds a feature requested by our clients in Vanga to remove lots with 0
quantity from the grid if you are doing a downwards adjustment.

Closes Third-Culture-Software#4062.
@jniles
Copy link
Collaborator Author

jniles commented Jan 15, 2020

@mbayopanda can I get a review?

Copy link
Collaborator

@mbayopanda mbayopanda left a comment

Choose a reason for hiding this comment

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

LGTM 👍

*/
function getVisibleLots(item) {
return item.lotsFull
.filter(v => (vm.adjustmentOption === 'decrease' ? v.quantity > 0 : true));
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

@mbayopanda
Copy link
Collaborator

bors r+

bors bot added a commit that referenced this pull request Jan 17, 2020
4097: feat(stock): rm 0 quantity in decrease adjustment r=mbayopanda a=jniles

Adds a feature requested by our clients in Vanga to remove lots with 0 quantity from the grid if you are doing a downwards adjustment.

Closes #4062.

This is what it looks like:

![emywtIplwW](https://user-images.githubusercontent.com/896472/72427394-f21a1980-378b-11ea-9f0b-b65438ad1ca2.gif)


Co-authored-by: Jonathan Niles <jonathanwniles@gmail.com>
@bors
Copy link
Contributor

bors bot commented Jan 17, 2020

Build failed

@jniles
Copy link
Collaborator Author

jniles commented Jan 17, 2020

bors r+

bors bot added a commit that referenced this pull request Jan 17, 2020
4097: feat(stock): rm 0 quantity in decrease adjustment r=jniles a=jniles

Adds a feature requested by our clients in Vanga to remove lots with 0 quantity from the grid if you are doing a downwards adjustment.

Closes #4062.

This is what it looks like:

![emywtIplwW](https://user-images.githubusercontent.com/896472/72427394-f21a1980-378b-11ea-9f0b-b65438ad1ca2.gif)


Co-authored-by: Jonathan Niles <jonathanwniles@gmail.com>
@bors
Copy link
Contributor

bors bot commented Jan 17, 2020

@bors bors bot merged commit 43c7a83 into Third-Culture-Software:master Jan 17, 2020
@jniles jniles deleted the feat-filter-stock-lots-for-zero-values-on-integration-decrease branch March 31, 2020 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stock adjustment should hide lots with 0 quantity when decreasing the value of stock
3 participants