Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[16.0] [MIG] account_cash_invoice: Migration to 16.0 #743

Merged
merged 19 commits into from
Jul 3, 2024
Merged

[16.0] [MIG] account_cash_invoice: Migration to 16.0 #743

merged 19 commits into from
Jul 3, 2024

Conversation

carlos-lopez-tecnativa
Copy link

  • refactor to new reconciliation process(no more cash.box.out, cash.box.in
  • no depends on account.bank.statement records, can use with account.bank.statement.line directly

@Tecnativa TT49793

@pedrobaeza
Copy link
Member

/ocabot migration account_cash_invoice

@pedrobaeza pedrobaeza changed the title [MIG] account_cash_invoice: Migration to 16.0 [16.0] [MIG] account_cash_invoice: Migration to 16.0 Jul 2, 2024
pedrobaeza
pedrobaeza reviewed Jul 2, 2024
View reviewed changes
account_cash_invoice/readme/CONTRIBUTORS.rst Outdated Show resolved Hide resolved
account_cash_invoice/wizard/cash_pay_invoice.py Outdated Show resolved Hide resolved
account_cash_invoice/wizard/cash_pay_invoice.py Outdated Show resolved Hide resolved
account_cash_invoice/wizard/cash_pay_invoice.py Outdated Show resolved Hide resolved
account_cash_invoice/wizard/cash_pay_invoice.xml Outdated Show resolved Hide resolved
account_cash_invoice/readme/DESCRIPTION.rst Outdated Show resolved Hide resolved
account_cash_invoice/readme/ROADMAP.rst Outdated Show resolved Hide resolved
account_cash_invoice/security/ir.model.access.csv Outdated Show resolved Hide resolved
@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Jul 2, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Jul 2, 2024
Open
14 tasks
carlos-lopez-tecnativa
carlos-lopez-tecnativa commented Jul 2, 2024
View reviewed changes
Copy link
Author

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pedrobaeza I have applied your suggestions. Could you please update your review?

pedrobaeza
pedrobaeza reviewed Jul 2, 2024
View reviewed changes
account_cash_invoice/models/account_move.py Outdated Show resolved Hide resolved
account_cash_invoice/readme/DESCRIPTION.rst Outdated Show resolved Hide resolved
account_cash_invoice/readme/DESCRIPTION.rst Outdated Show resolved Hide resolved
account_cash_invoice/wizard/cash_pay_invoice.py Outdated Show resolved Hide resolved
@carolinafernandez-tecnativa
Copy link

Please squash administrative commits.
Functional review LGTM.

emagdalenaC2i and others added 15 commits July 3, 2024 06:03
@emagdalenaC2i @carlos-lopez-tecnativa
[11.0] [FIX] Category in account_cash_invoice (#1)
473942d 
This is a minor fix in the manifest.py file of account_cash_invoice:

Change category from "Account" to "Accounting" in order to follow the categories set in Odoo 11
Change the website to the repository web https://github.com/OCA/account-payment
Remove the lines # -- coding: utf-8 --
@mamcode @carlos-lopez-tecnativa
[MIG] account_cash_invoice: Migration to 12.0
de6fbba
@carlos-lopez-tecnativa
[12.0][FIX] Continue Migration
690b2f4
@pedrocs-exo @carlos-lopez-tecnativa
Added translation using Weblate (Portuguese)
680c1ee
@badbole @carlos-lopez-tecnativa
Added translation using Weblate (Croatian)
8ece1dd
@legalsylvain @carlos-lopez-tecnativa
[FIX] account_cash_invoice : bad domain in xml view generate error in…
e1e5853 
… many journals context
@acsonefho @carlos-lopez-tecnativa
[IMP] account_cash_invoice: black, isort, prettier
38096bf
@acsonefho @etobella @carlos-lopez-tecnativa
[13.0][MIG] account_cash_invoice
83323ee 
Co-authored-by: Enric Tobella <etobella@creublanca.es>
@etobella @carlos-lopez-tecnativa
[IMP] account_cash_invoice: Add migration script
69f536d
@acsonefho @carlos-lopez-tecnativa
[IMP] account_cash_invoice: black, isort, prettier
8051597
@acsonefho @carlos-lopez-tecnativa
[14.0][MIG] account_cash_invoice from 13.0 to 14.0
6a3fe7a
@carlos-lopez-tecnativa
[CHG] account_cash_invoice: remove useless code + limit to unpaid inv…
9b80720 
…oices
@olgamarcocb @carlos-lopez-tecnativa
[FIX] account_cash_invoice: Fixed button and lines
f3fe276
@olgamarcocb @carlos-lopez-tecnativa
Added translation using Weblate (Spanish)
6216ef2
@carlos-lopez-tecnativa
[UPD] add index on invoice_id to speed up performance on big databases
001eb24
@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-743-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 417c660 into OCA:16.0 Jul 3, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 61e6cd2. Thanks a lot for contributing to OCA. ❤️

@carlos-lopez-tecnativa carlos-lopez-tecnativa deleted the 16.0-mig-account_cash_invoice branch July 3, 2024 14:08
@carlos-lopez-tecnativa carlos-lopez-tecnativa mentioned this pull request Jul 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pedrobaeza pedrobaeza pedrobaeza approved these changes

@carolinafernandez-tecnativa carolinafernandez-tecnativa carolinafernandez-tecnativa approved these changes

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.