Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0047492 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [POS2] POS | minor | have not tried | 2021-08-03 13:23 | 2021-12-13 14:15 | |||
Reporter | cberner | View Status | public | |||||
Assigned To | elara | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | |||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0047492: Refreshing browser in a Product Customization process will leave it inconsistent, not completed | |||||||
Description | Product 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 Reproduce | Install 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 Solution | On refresh if the product configuration process was open, reopen it on the same step. | |||||||
Tags | No 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 |