[GitHub] [ofbiz-plugins] lodhiravi opened a new pull request #41: Fixed: Add To Cart and related UI issue for the products with variant explosion and feature explosion (OFBIZ-11970)
Locked
 
|
lodhiravi opened a new pull request #41: URL: https://github.com/apache/ofbiz-plugins/pull/41 Done following- 1. Corrected markup to fix the issue where add to cart is not working for the feature explosion products. 2. Added a separate screen for rendering Product UOM dropdown and set the last view correctly to avoid ajax view to be used in subsequent request (OFBIZ-11970). ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [hidden email] |
|
|
In reply to this post by GitBox
mbrohl commented on pull request #41: URL: https://github.com/apache/ofbiz-plugins/pull/41#issuecomment-778643107 @lodhiravi Thanks for your appreciated contribution. Can you explain why we need an additional screen in the ecommerce component? Thanks! ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [hidden email] |
|
|
In reply to this post by GitBox
lodhiravi commented on pull request #41: URL: https://github.com/apache/ofbiz-plugins/pull/41#issuecomment-817094416 > @lodhiravi Thanks for your appreciated contribution. > Can you explain why we need an additional screen in the ecommerce component? Thanks! There are tw reasons to intro -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [hidden email] |
|
|
In reply to this post by GitBox
lodhiravi closed pull request #41: URL: https://github.com/apache/ofbiz-plugins/pull/41 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [hidden email] |
|
|
In reply to this post by GitBox
lodhiravi edited a comment on pull request #41: URL: https://github.com/apache/ofbiz-plugins/pull/41#issuecomment-817094416 @mbrohl Sorry for the late response. There are two reasons (after my recent push) to introduce a new screen - 1. To Set the last saved view in the last view to avoid ajax request's view being used in the subsequent requests. Earlier when we select color/size an ajax request to render ProductUomDropDownOnly used and when we select uom and clicks on add to cart, the item gets added but the ProductUomDropDownOnly gets rendered as the last view was to render that. 2. To improve the drop-down markup using bootstrap in order to make it consistent with other front-end screens. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [hidden email] |
«
Return to OFBiz - Notifications
|
1 view|%1 views
| Free forum by Nabble | Edit this page |