[16.0][MIG] pos_customer_wallet: migration to 16.0 (continue) #353

legalsylvain · 2025-03-10T06:51:39Z

Do not review for the time being.

Ajoute un bandeau pour indiquer que le compte client est utilisable.

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

….config Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

…t payment method Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

…stead of settings Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

…ines This does not yet live-update in the POS. The window needs to be reloaded to see the updated balance. Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

… balance Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

…ner._invoice_total algorithm Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

…r account is set instead Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

…ly tree Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

…thout perms Previously there was a permission error on trying to read the `account.bank.statement.line` model. Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

…m in PoS Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

…member of 'Point of Sale / User' group Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

The class name was wrongly copied from some other place and does not reflect what it's trying to represent. Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

base_suspend_security is not available in Odoo 16. Let's see if we can do without. Clean up this commit later.

This change was also done in account_customer_wallet. It simplifies things. This change is necessary for the 16.0 migration, but is not included in its commit for clarity's sake. Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

…t computation The pos session does not create these anymore. It creates pos.payment records instead. Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

codecov-commenter · 2025-03-10T06:55:10Z

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 84.82143% with 17 lines in your changes missing coverage. Please review.

Project coverage is 79.07%. Comparing base (110387a) to head (df8b56e).
Report is 88 commits behind head on 16.0.

Files with missing lines	Patch %	Lines
pos_customer_wallet/models/pos_session.py	40.00%	9 Missing ⚠️
pos_customer_wallet/models/pos_config.py	58.33%	5 Missing ⚠️
account_customer_wallet/models/product_template.py	40.00%	2 Missing and 1 partial ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@             Coverage Diff             @@
##             16.0     #353       +/-   ##
===========================================
- Coverage   94.17%   79.07%   -15.11%     
===========================================
  Files          34       87       +53     
  Lines         584     1214      +630     
  Branches       56      120       +64     
===========================================
+ Hits          550      960      +410     
- Misses         27      240      +213     
- Partials        7       14        +7

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…figure wallet account at product level, if the product is flagged as 'Customer Wallet Product'

…on that it is possible to use Wallet Journal. - Add same banner on payment wizard

carmenbianca and others added 30 commits March 10, 2025 07:49

[ADD] pos_customer_wallet

085c48d

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] pos_customer_wallet: Display wallet balance on POS payment screen

765e096

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] pos_customer_wallet: Make global module toggle available in pos…

b3579ab

….config Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] pos_customer_wallet: Journal demo data

56f163a

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] pos_customer_wallet: Mandate selected customer when using walle…

5011d31

…t payment method Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] pos_customer_wallet: Check sufficient balance before POS purchase

d1dd4a6

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] account_customer_wallet: Write account_wallet to res.company in…

72c0360

…stead of settings Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] pos_account_wallet: Update wallet balance in POS

3d2a38a

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] pos_customer_wallet: Update write_date on computation of wallet…

c615a34

… balance Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[REM] account_customer_wallet: Remove some fields and api.depends lines

b473c95

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] pos_customer_wallet: Tests

1a673ca

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[REF] pos_customer_wallet: Re-implement wallet computation using part…

2aa883b

…ner._invoice_total algorithm Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[REM] account_customer_wallet: Custom wallet toggle; depend on whethe…

b92518d

…r account is set instead Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] account_customer_wallet: Search balance on all partners in fami…

0c4c71a

…ly tree Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] pos_customer_wallet: Avoid null error when no customer is selected

1401fbd

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] account_customer_wallet: README.rst

079b3c6

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[UPD] Update pos_customer_wallet.pot

fb776ec

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

pos_customer_wallet 12.0.1.0.1

642121c

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] SCRLfs → SC

5e57552

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] pos_customer_wallet: Show balance on ticket

5b3b0d3

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] pos_customer_wallet: typo

acccb35

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[REF] *_customer_wallet: Create partner.get_all_partners_in_family

ce8d94c

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] pos_customer_wallet_is_user

5a3c8ba

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[ADD] *_customer_wallet: Change log entries

a252e07

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[UPD] Update pos_customer_wallet.pot

35c619b

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[UPD] changelog

6e708b0

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[UPD] README.rst

bcb7ff9

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

pos_customer_wallet 12.0.1.1.0

0a307b8

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

legalsylvain and others added 24 commits March 10, 2025 07:49

[REF] pos_customer_wallet : create a function is_balance_above_minimu…

8d082b2

…m in PoS Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] allow user to access to the wallet amount even if they are not …

fd86c70

…member of 'Point of Sale / User' group Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[UPD] Update pos_customer_wallet.pot

dbd9b46

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[UPD] README.rst

eabb2ea

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] pos_customer_wallet[_partner_is_user]: Pass required argument

33398ee

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

pos_customer_wallet 12.0.2.1.0

f49a4e5

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] pos_customer_wallet[_*]: Rename foodprint class

49a5979

The class name was wrongly copied from some other place and does not reflect what it's trying to represent. Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be> Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] pos_customer_wallet_[*]: Repair maintainer tags in manifest

0543a1a

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[UPD] Run prettier

e6c09e3

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[UPD] README.rst

13ee7aa

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

pos_customer_wallet 12.0.2.1.1

65d7b72

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[UPD] Update pos_customer_wallet.pot

e34dd76

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[IMP] pos_customer_wallet: black, isort, prettier

a5df49c

[MIG] pos_customer_wallet: Migration to 13.0

21a2f5a

[MIG] pos_customer_wallet: Migration to 14.0

1fa7ba0

[MIG] pos_customer_wallet: Migration to 15.0

454a283

[MIG] pos_customer_wallet: Migration to 16.0

9434388

[FIX] pos_customer_wallet: Remove dependency on base_suspend_security

c145f7c

base_suspend_security is not available in Odoo 16. Let's see if we can do without. Clean up this commit later.

[REM] pos_customer_wallet: Remove bank statement line method of walle…

8c70c5b

…t computation The pos session does not create these anymore. It creates pos.payment records instead. Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] pos_customer_wallet: Run pre-commit + satisfy eslint

a27cd3c

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

[FIX] pos_customer_wallet: Add some copyright headers

6d39eba

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

fixup! [MIG] pos_customer_wallet: Migration to 16.0

3dee566

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

fixup! [MIG] pos_customer_wallet: Migration to 16.0

deca7d4

Signed-off-by: Carmen Bianca BAKKER <carmen@coopiteasy.be>

legalsylvain added 3 commits March 10, 2025 08:50

[REF] *_customer_wallet: run pre-commit with latest version

71332b2

[IMP] account_customer_wallet: Avoid misconfiguration. No need to con…

7163ed0

…figure wallet account at product level, if the product is flagged as 'Customer Wallet Product'

[IMP] account_customer_wallet: add a banner on payment form, to menti…

df8b56e

…on that it is possible to use Wallet Journal. - Add same banner on payment wizard

legalsylvain changed the title ~~16.0 mig pos customer wallet continue~~ 16.0 mig pos_customer_wallet continue Mar 28, 2025

huguesdk changed the title ~~16.0 mig pos_customer_wallet continue~~ [16.0][MIG] pos_customer_wallet: migration to 16.0 (continue) Nov 3, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[16.0][MIG] pos_customer_wallet: migration to 16.0 (continue) #353

[16.0][MIG] pos_customer_wallet: migration to 16.0 (continue) #353

Uh oh!

legalsylvain commented Mar 10, 2025 •

edited

Loading

Uh oh!

codecov-commenter commented Mar 10, 2025 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

[16.0][MIG] pos_customer_wallet: migration to 16.0 (continue) #353

Are you sure you want to change the base?

[16.0][MIG] pos_customer_wallet: migration to 16.0 (continue) #353

Uh oh!

Conversation

legalsylvain commented Mar 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

codecov-commenter commented Mar 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

legalsylvain commented Mar 10, 2025 •

edited

Loading

codecov-commenter commented Mar 10, 2025 •

edited

Loading