Openbravo Issue Tracking System - Modules | |||||
View Issue Details | |||||
ID | Project | Category | View Status | Date Submitted | Last Update |
0050295 | Modules | Loyalty Programs | public | 2022-09-20 21:59 | 2022-09-20 21:59 |
Reporter | axelmercado | ||||
Assigned To | Retail | ||||
Priority | high | Severity | major | Reproducibility | N/A |
Status | new | Resolution | open | ||
Platform | OS | 5 | OS Version | ||
Product Version | |||||
Target Version | Fixed in Version | ||||
Merge Request Status | |||||
Regression date | |||||
Regression introduced by commit | |||||
Regression level | |||||
Review Assigned To | |||||
Support ticket | |||||
OBNetwork customer | |||||
Regression introduced in release | |||||
Summary | 0050295: Loyalty: point fields with decimals | ||||
Description | Patch proposed by the partner Practics: Our client needs to have the points with decimals, at the end his calculation rules are based on a % of the amount of the sale. For example, 1% points on a sale of 50 euros --> 0.5 points. The case is that by the design of loyalty the fields are integers in ad_column We have generated a patch that affects several modules to make the change from Long to BigDecimal. We would like you to analyze it and look to contribute. We think that for any customer it is valid at the end to do the management with decimals should not be a problem in any customer. One point that you have to take into account is that if other modules that you have set a Long you will have to adjust them. | ||||
Steps To Reproduce | N/A | ||||
Proposed Solution | Attached patch | ||||
Additional Information | |||||
Tags | No tags attached. | ||||
Relationships | |||||
Attached Files | puntosConDecimales.diff (36,510) 2022-09-20 21:59 https://issues.openbravo.com/file_download.php?file_id=17535&type=bug | ||||
Issue History | |||||
Date Modified | Username | Field | Change | ||
2022-09-20 21:59 | axelmercado | New Issue | |||
2022-09-20 21:59 | axelmercado | Assigned To | => Retail | ||
2022-09-20 21:59 | axelmercado | File Added: puntosConDecimales.diff |
There are no notes attached to this issue. |