Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0047492
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[POS2] POSminorhave not tried2021-08-03 13:232021-12-13 14:15
ReportercbernerView Statuspublic 
Assigned Toelara 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0047492: Refreshing browser in a Product Customization process will leave it inconsistent, not completed

DescriptionProduct customization process, like for example, adding a beeper to a product, and refreshing in the process results in it not being configured properly, requiring the user to know and edit it manually previous to paying the ticket.

This should not happen, refreshing should not allow leaving the window, the same as clicking on other options while in a product configuration process (changing to other windows or flows is blocked, while configuring a product).
Steps To ReproduceInstall this demo product configuration module:
https://gitlab.com/openbravo/demo/org.openbravo.pos2.demo.productconfig [^]

After installing:
1. Open WebPOS
2. Add one of these products(any): WVG/M0019','OBREST_Soup','OBREST_GarlicBread','OBREST_Beer
3. Leave the process by refreshing(F5)
Check that the product has not been completely configured, so you should not be able to pay, and requires editing the product configuration (click on product, and on popover on the button to edit configuration).
Proposed SolutionOn refresh if the product configuration process was open, reopen it on the same step.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0131967)
cberner (developer)
2021-09-21 18:21

There's another option instead of opening the previous step after a refresh.

Some ticketline products may be marked somehow to show to the user these are pending to configure some modifiers. I would confirm this with David Baz, but I think from a UX perspective it would be better to have them marked instead of reopenning some step you wouldn't expect.
(0132690)
hgbot (developer)
2021-10-28 03:55

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/721 [^]
(0132691)
hgbot (developer)
2021-10-28 03:57

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2.restaurant/-/merge_requests/90 [^]
(0133652)
hgbot (developer)
2021-12-13 14:15

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: 96a57d3daae3bdba8f01841a141a92a60258cb8c
Author: Eduardo Lara <eduardo.lara@smfconsulting.es>
Date: 13-12-2021 13:15:23
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/96a57d3daae3bdba8f01841a141a92a60258cb8c [^]

Fixed ISSUE-47492: Added Missing Configuration label to configurable products

This marks a product as missing configuration if for some reason, not all required modifiers have been added.

---
M web-jspack/org.openbravo.pos2/src/components/ProofOfPayment/ProofOfPaymentDetails/ProofOfPaymentDetailLine/__test__/ProofOfPaymentDetailLine.test.jsx
M web-jspack/org.openbravo.pos2/src/components/ProofOfPayment/ProofOfPaymentDetails/__test__/ProofOfPaymentDetails.test.jsx
M web-jspack/org.openbravo.pos2/src/components/SelectLines/SelectedLinesGrid/__test__/SelectLines.test.jsx
M web-jspack/org.openbravo.pos2/src/components/Ticket/TicketArea/__test__/TicketArea.test.jsx
M web-jspack/org.openbravo.pos2/src/components/Ticket/TicketLines/ProductConfiguration.jsx
M web-jspack/org.openbravo.pos2/src/components/Ticket/TicketLines/TicketLineSchemaLabel/TicketLineSchemaLabel.scss
M web-jspack/org.openbravo.pos2/src/components/Ticket/TicketLines/__test__/TicketLines.test.jsx
M web-jspack/org.openbravo.pos2/src/components/TicketLineRenderCell/__test__/TicketLineRenderCell.test.jsx
M web-jspack/org.openbravo.pos2/src/components/VerifiedReturns/VerifiedReturnsGrid/__test__/VerifiedReturnsGrid.test.jsx
---
(0133653)
hgbot (developer)
2021-12-13 14:15

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/721 [^]
(0133654)
hgbot (developer)
2021-12-13 14:15

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2.restaurant [^]
Changeset: e1550cb535cf1fee6600e96bf01e6063e69ab740
Author: Eduardo Lara <eduardo.lara@smfconsulting.es>
Date: 13-12-2021 13:15:33
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2.restaurant/-/commit/e1550cb535cf1fee6600e96bf01e6063e69ab740 [^]

Fixed ISSUE-47492: Added Missing Configuration label to configurable products

---
M web-jspack/org.openbravo.pos2.restaurant/src/components/TicketMenuLine/TicketMenuInProgressLineItem/TicketMenuInProgressLineItem.jsx
M web-jspack/org.openbravo.pos2.restaurant/src/components/TicketMenuLine/TicketMenuInProgressLineItem/__test__/TicketMenuInProgressLineItem.test.jsx
M web-jspack/org.openbravo.pos2.restaurant/src/components/TicketMenuLine/TicketMenuInProgressLineItems/__test__/TicketMenuInProgressLineItems.test.jsx
M web-jspack/org.openbravo.pos2.restaurant/src/components/TicketMenuLine/TicketMenuLineItems/TicketMenuLineItems.jsx
M web-jspack/org.openbravo.pos2.restaurant/src/components/TicketMenuLine/TicketMenuLineItems/__test__/TicketMenuLineItems.test.jsx
M web-jspack/org.openbravo.pos2.restaurant/src/components/TicketMenuLine/__test__/TicketMenuLine.test.jsx
---
(0133655)
hgbot (developer)
2021-12-13 14:15

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2.restaurant/-/merge_requests/90 [^]

- Issue History
Date Modified Username Field Change
2021-08-03 13:23 cberner New Issue
2021-08-03 13:23 cberner Assigned To => cberner
2021-08-03 13:23 cberner Triggers an Emergency Pack => No
2021-08-04 11:36 dmiguelez Resolution time => 1630447200
2021-08-04 11:36 dmiguelez Status new => acknowledged
2021-08-27 14:32 cberner Resolution time 1630447200 => 1632952800
2021-09-21 18:21 cberner Note Added: 0131967
2021-09-28 03:27 elara Assigned To cberner => elara
2021-09-30 15:49 AugustoMauch Status acknowledged => scheduled
2021-10-28 03:55 hgbot Note Added: 0132690
2021-10-28 03:57 hgbot Note Added: 0132691
2021-12-13 14:15 hgbot Resolution open => fixed
2021-12-13 14:15 hgbot Status scheduled => closed
2021-12-13 14:15 hgbot Note Added: 0133652
2021-12-13 14:15 hgbot Note Added: 0133653
2021-12-13 14:15 hgbot Note Added: 0133654
2021-12-13 14:15 hgbot Note Added: 0133655


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker