feat: allow user to bulk upload items with quantity to shopping cart

agritheory / inventory_tools

A collection of features to streamline and enhance inventory management and manufacturing workflows in ERPNext.

https://agritheory.com/documentation/inventory_tools

Other

24 stars 13 forks source link

feat: allow user to bulk upload items with quantity to shopping cart #70

Closed bhattdevarsh closed 7 months ago

bhattdevarsh commented 7 months ago

https://github.com/agritheory/inventory_tools/issues/54

New page /bulk-order
Allow users to add item and quantity line by line
Allow users to add item and quantity as a bulk
Process data and use existing update_cart def to update cart items.

Possible Scope:

[x] Improvise UX
[x] Enqueue update_cart to handle large data

bulk_order

bhattdevarsh commented 7 months ago

@agritheory @bhattdevarsh Is there a way to drop a Vue file into the www folder? Would that just work out of the box in version-14? Or would it require version-15's Vue 3 dependency?

I think we had somewhat similar conversation before starting and concluded that we may not need Vue and just normal web page should do the job.

bhattdevarsh commented 7 months ago

@Alchez

New Update:

Removed line-by-line form.
Improvised code.
Enqueued the update_cart def

Point of Discussion: Exception handling not implemented at this stage: The form action is redirecting the user to python function, in case of any error on python function will show the traceback page instead of a popup. If that is fine, we can continue or else we can bring back the .js file which can handle the callback/error.

Alchez commented 7 months ago

@bhattdevarsh this looks good to me. Is this PR ready to be merged, or are you adding something else as well?

bhattdevarsh commented 7 months ago

@bhattdevarsh this looks good to me. Is this PR ready to be merged, or are you adding something else as well?

good from my end.

Alchez commented 7 months ago

@agritheory @bhattdevarsh the test failure seems to be related to an error upstream with OS resolution. There's an issue created for it over at https://github.com/actions/runner-images/issues/7210.