Openbravo Issue Tracking System - Retail Modules |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0036817 | Retail Modules | Web POS | public | 2017-09-08 19:38 | 2017-09-27 16:52 |
|
| Reporter | dbaz | |
| Assigned To | ranjith_qualiantech_com | |
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | RR17Q4 | |
| Merge Request Status | |
| Review Assigned To | marvintm |
| OBNetwork customer | No |
| Support ticket | |
| Regression level | |
| Regression date | |
| Regression introduced in release | |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0036817: [UX] Approval popup is hidden when an error happens |
| Description | There could be several situations where an error can happen while trying to introduce the approval credentials:
* Bad password
* Empty password
* Not allowed user
In all of these cases, the approval popup gets closed once the error message it is shown.
It is a pain, since the user should again do all the required actions to invoke this popup to introduce proper credentials.
Also, the error message, instead of be shown as a warning or error in the top-right corner, it should be shown within the same popup in order to improve its usability. NOTE: If there is any kind of these errors, both inputs (username and password) must be erased, and the focus should be set in the 'username' input. |
| Steps To Reproduce | Create an approval (for example 'WebPOS Cash up window') and once it be shown, try to enter:
* Bad password
* Empty password
* Not allowed user |
| Proposed Solution | The popup should remain visible to allow the user enter new credentials.
I attach some mockups of the styling that the errors should have. |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | | related to | design defect | 0036816 | | closed | Retail | [UX] In case of error in Approval, it should display a Popup instead of the top-right error message |
|
| Attached Files |  ApprovalError_01.png (74,734) 2017-09-13 20:15
https://issues.openbravo.com/file_download.php?file_id=11060&type=bug
ApprovalError_02.png (71,771) 2017-09-13 20:16
https://issues.openbravo.com/file_download.php?file_id=11061&type=bug
ApprovalError_03.png (73,223) 2017-09-13 20:16
https://issues.openbravo.com/file_download.php?file_id=11062&type=bug
|
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2017-09-08 19:38 | dbaz | New Issue | |
| 2017-09-08 19:38 | dbaz | Assigned To | => Retail |
| 2017-09-08 19:38 | dbaz | OBNetwork customer | => No |
| 2017-09-08 19:38 | dbaz | Triggers an Emergency Pack | => No |
| 2017-09-08 19:38 | dbaz | Relationship added | related to 0036816 |
| 2017-09-13 20:15 | dbaz | Description Updated | bug_revision_view_page.php?rev_id=15894#r15894 |
| 2017-09-13 20:15 | dbaz | Proposed Solution updated | |
| 2017-09-13 20:16 | dbaz | File Added: ApprovalError_01.png | |
| 2017-09-13 20:16 | dbaz | File Added: ApprovalError_02.png | |
| 2017-09-13 20:16 | dbaz | File Added: ApprovalError_03.png | |
| 2017-09-14 12:53 | ranjith_qualiantech_com | Assigned To | Retail => ranjith_qualiantech_com |
| 2017-09-14 12:53 | ranjith_qualiantech_com | Status | new => scheduled |
| 2017-09-20 08:05 | hgbot | Checkin | |
| 2017-09-20 08:05 | hgbot | Note Added: 0099156 | |
| 2017-09-20 08:06 | hgbot | Checkin | |
| 2017-09-20 08:06 | hgbot | Note Added: 0099158 | |
| 2017-09-20 08:06 | hgbot | Status | scheduled => resolved |
| 2017-09-20 08:06 | hgbot | Resolution | open => fixed |
| 2017-09-20 08:06 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8def4fd328e0cfbce04417bc3e6a21ce7f3ef2ea [^] |
| 2017-09-27 16:51 | hgbot | Checkin | |
| 2017-09-27 16:51 | hgbot | Note Added: 0099635 | |
| 2017-09-27 16:52 | hgbot | Checkin | |
| 2017-09-27 16:52 | hgbot | Note Added: 0099636 | |
| 2017-09-27 16:52 | marvintm | Review Assigned To | => marvintm |
| 2017-09-27 16:52 | marvintm | Status | resolved => closed |
| 2017-09-27 16:52 | marvintm | Fixed in Version | => RR17Q4 |
|
Notes |
|
|
(0099156)
|
|
hgbot
|
|
2017-09-20 08:05
|
|
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 57ba410945404d14852eb358e0b47042833e58b6
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Wed Sep 20 11:16:07 2017 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/57ba410945404d14852eb358e0b47042833e58b6 [^]
Related to issue 36817 : Changes in Approval Credential Validations
* If the approval is failed, error should be shown in approval popup itself instead of closing it
---
M web/org.openbravo.mobile.core/assets/css/ob-standard.css
M web/org.openbravo.mobile.core/source/component/obpos-approval.js
---
|
|
|
|
(0099158)
|
|
hgbot
|
|
2017-09-20 08:06
|
|
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 8def4fd328e0cfbce04417bc3e6a21ce7f3ef2ea
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Wed Sep 20 11:16:05 2017 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8def4fd328e0cfbce04417bc3e6a21ce7f3ef2ea [^]
Fixed issue 36817 : Changes in CheckApproval Validations
* If the approval is failed, error should be shown in approval popup itself
---
M src-db/database/sourcedata/AD_MESSAGE.xml
M src/org/openbravo/retail/posterminal/utility/CheckApproval.java
M web/org.openbravo.retail.posterminal/js/utils/ob-utilitiesuipos.js
---
|
|
|
|
(0099635)
|
|
hgbot
|
|
2017-09-27 16:51
|
|
|
|
|
(0099636)
|
|
hgbot
|
|
2017-09-27 16:52
|
|
|