agritheory / beam

General Purpose 2D barcode scanning for ERPNext
https://agritheory.com/documentation/beam/
Other
23 stars 11 forks source link

Descendant warehouse lookup with warehouse types #132

Closed MyuddinKhatri closed 2 months ago

MyuddinKhatri commented 3 months ago

closes #107

agritheory commented 3 months ago

@MyuddinKhatri The description language isn't clear to me - does this exclude the selected warehouse types or include only the selected warehouse types?

If the logic is exclude - "These Warehouse Types will be excluded when checking for inventory availability"

MyuddinKhatri commented 3 months ago

@MyuddinKhatri The description language isn't clear to me - does this exclude the selected warehouse types or include only the selected warehouse types?

If the logic is exclude - "These Warehouse Types will be excluded when checking for inventory availability"

@agritheory As it excludes I'll change it to the text you've mentioned above.

MyuddinKhatri commented 3 months ago

@agritheory As discussed will implement - show quantities in the descendant warehouses with indentation in the report.

MyuddinKhatri commented 3 months ago

@agritheory This is how it looks,

image right now it is showing flat tree (parent -> all nested children in one indent), do we want to show exact tree structure for nested children? One more thing, what do you think about data for Net req qty in child warehouses?

agritheory commented 3 months ago

do we want to show exact tree structure for nested children?

I think showing the demand:location relationship is enough.

What do you think about data for Net req qty in child warehouses?

We should show required quantity and UOM from specific WO/SO and total demand from all WO/SO. This report isn't likely to be the primary interface for this feature, it will be more of a debugging and analysis tool.

MyuddinKhatri commented 3 months ago

Added Stock UOM column in the report image