Skip to content

Commit

Permalink
Merge branch '14.0' into qtiplb-patch-1
Browse files Browse the repository at this point in the history
  • Loading branch information
majouda authored Jan 2, 2025
2 parents 1857827 + cc14896 commit 85829bd
Show file tree
Hide file tree
Showing 58 changed files with 302 additions and 287 deletions.
6 changes: 3 additions & 3 deletions .docker_files/main/__manifest__.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
"category": "Other",
"summary": "Install all addons required for testing.",
"depends": [
"account_accountant", # Used for testing account_fr_ca_labels
"l10n_generic_coa", # Used for testing addons
"account_additional_group",
"account_analytic_required_forbidden",
Expand All @@ -20,6 +19,7 @@
"account_closing_journal",
"account_closing_journal_mis_builder",
"account_closing_wizard",
"account_financial_report", # Used for testing account_fr_ca_labels
"account_fiscalyear_end_on_company",
"account_fr_ca_labels",
"account_invoice_constraint_chronology_forced",
Expand All @@ -33,7 +33,7 @@
"account_payment_term_usage_purchase",
"account_payment_term_usage_sale",
"account_payment_widget_link",
"account_report_line_menu",
"account_reconciliation_widget", # Used for testing account_fr_ca_labels
"account_report_trial_balance",
"account_search_by_amount",
"account_show_full_features",
Expand Down Expand Up @@ -61,7 +61,7 @@
"old_accounts",
"payment_list_not_sent",
"payment_stripe_not_silenced",
"account_sale_invoice_date_required"
"account_sale_invoice_date_required",
],
"installable": True,
}
2 changes: 1 addition & 1 deletion .docker_files/odoo.conf
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ test_report_directory = False
translate_modules = ['all']
unaccent = False
without_demo = False
workers = 2
workers = 0
xmlrpc = True
xmlrpc_interface =
xmlrpc_port = 8069
Expand Down
1 change: 0 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ COPY account_payment_term_usage /mnt/extra-addons/account_payment_term_usage
COPY account_payment_term_usage_purchase /mnt/extra-addons/account_payment_term_usage_purchase
COPY account_payment_term_usage_sale /mnt/extra-addons/account_payment_term_usage_sale
COPY account_payment_widget_link /mnt/extra-addons/account_payment_widget_link
COPY account_report_line_menu /mnt/extra-addons/account_report_line_menu
COPY account_report_trial_balance /mnt/extra-addons/account_report_trial_balance
COPY account_sale_invoice_date_required /mnt/extra-addons/account_sale_invoice_date_required
COPY account_search_by_amount /mnt/extra-addons/account_search_by_amount
Expand Down
2 changes: 1 addition & 1 deletion account_check_deposit_enhanced/__manifest__.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

{
"name": "Account Check Deposit Enhanced",
"version": "1.0.0",
"version": "1.1.0",
"category": "Accounting",
"license": "AGPL-3",
"summary": "Enhance check deposits",
Expand Down
11 changes: 6 additions & 5 deletions account_check_deposit_enhanced/models/account_check_deposit.py
Original file line number Diff line number Diff line change
@@ -1,14 +1,15 @@
# © 2022 Numigi (tm) and all its contributors (https://bit.ly/numigiens)
# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).

from odoo import api, models
from odoo import models


class AccountCheckDeposit(models.Model):
_inherit = "account.check.deposit"

@api.model
def _prepare_counterpart_move_lines_vals(self, *args, **kwargs):
vals = super()._prepare_counterpart_move_lines_vals(*args, **kwargs)
vals["partner_id"] = self.company_id.partner_id.id
def _prepare_move_vals(self):
vals = super()._prepare_move_vals()
for line in vals["line_ids"]:
if 'debit' in line[2].keys():
line[2]['partner_id'] = self.company_id.partner_id.id
return vals
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,9 @@ def setUpClass(cls):
}
)

def test_debit_move_line_has_company(self):
def test_deposit_line_ids_partner_id(self):
self.deposit.validate_deposit()
debit = self.deposit.line_ids.filtered(lambda l: l.debit)
assert debit.partner_id == self.env.user.company_id.partner_id
assert debit.partner_id == self.deposit.company_id.partner_id
credit = self.deposit.line_ids.filtered(lambda l: l.credit)
assert credit.partner_id == self.partner
assert not credit.partner_id
20 changes: 0 additions & 20 deletions account_fr_ca_labels/tests/test_fr_translations.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,26 +34,6 @@ def test_translations_found_with_correct_term(self, data):
assert self._find_translation(data[0], data[1])


@ddt
class TestAgedBalance(TranslationCase):

@data(
('Aged Receivable', 'Balance agée clients'),
('Aged Payable', 'Balance agée fournisseurs'),
('Aged Partner Balances', 'Balances agées des tiers'),
)
def test_no_translation_found_with_wrong_term(self, data):
assert not self._find_translation(data[0], data[1])

@data(
('Aged Receivable', 'Âge des comptes clients'),
('Aged Payable', 'Âge des comptes fournisseurs'),
('Aged Partner Balances', 'Âge des comptes'),
)
def test_translations_found_with_correct_term(self, data):
assert self._find_translation(data[0], data[1])


@ddt
class TestReconciliation(TranslationCase):

Expand Down
68 changes: 25 additions & 43 deletions canada_bank_transfer/README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,11 @@ This module enables to generate credit transfer files for transactions between b

Configuration
-------------
As member of `Accounting / Manager`, I go to the list view of accounting journals.

.. image:: static/description/journal_list.png
As member of `Invoicing / Billing Administrator`, I go to the list view of journals (`Configuration / Journals`).

I open the form view of my bank journal.

In the 'Advanced Settings' tab, I see a checkbox `EFT`.
In the `Payements Configuration` tab, I see a checkbox `EFT`.

.. image:: static/description/journal_form_eft_checkbox.png

Expand All @@ -28,6 +26,10 @@ User Short Name
This is a short version of your company name.
It must be composed of maximum 15 alphanumeric caracters.

User Long Name
~~~~~~~~~~~~~~~
This is a long version of your company name.

User Number
~~~~~~~~~~~
This number is attributed by your bank to identify your company.
Expand Down Expand Up @@ -67,23 +69,19 @@ Therefore, when generating a new EFT, this number will be used.

Bank Account Configuration
--------------------------
Next, I go to the `Bank Account` tab.
Next, I go to the `Jounal Entries` tab.

This tab contains the information related to my company's bank account.

.. image:: static/description/journal_form_bank_account_tab.png

In the field `Bank Account`, I click on "Create and Edit".

.. image:: static/description/journal_form_bank_account_field.png
In the field `Account Number`, I create a bank account.

I fill my bank account number. It is composed from 7 to 12 digits.

.. image:: static/description/bank_account_number.png

In the field `Bank`, I click on "Create and Edit".

.. image:: static/description/bank_account_bank_field.png
In the field `Bank`, I create a bank.

I fill the name of the bank and its institution number, then I click on `Save`.

Expand All @@ -99,28 +97,26 @@ The configuration of my journal is now complete, I click on `Save`.

Preparing the Payments
----------------------
As member of `Accounting / Billing`, I go to `Accounting / Vendors / Payments`.
As member of `Invoicing / Billing`, I go to `Invoicing / Vendors / Payments`.
Then I click on `Create`.

.. image:: static/description/vendor_payment_list.png

I fill the partner and the payment amount. I check `EFT` as payment method.

.. image:: static/description/payment_form.png

I click on `Save`, then I click on `Confirm`.

.. image:: static/description/payment_form_confirm.png

The payment is now `Posted`. At this stage, it is ready to be selected for an EFT transfer.

.. image:: static/description/payment_form_posted.png
Since version 1.3.0 of this module:
A new filter `Not Sent` is added in payments.

.. image:: static/description/payment_not_sent_filter.png

Preparing the EFT
-----------------
Once I have multiple EFT payments posted, I go back to the list of vendor payments.

I check both the `Posted` and `EFT` filters.
I check both the `Posted`, `Not Sent` and `EFT` filters.

.. image:: static/description/vendor_payment_list_filtered.png

Expand All @@ -130,8 +126,6 @@ I select my payments and click on `Generate EFT` in the action menu.

A draft EFT is created.

.. image:: static/description/eft_draft.png

For each payment, I select the recipient bank account.

If the bank account is not already defined for a given partner, I may create and edit a new one.
Expand All @@ -152,52 +146,41 @@ The bank must have an institution number (3 digits).

Once all the destination bank accounts are selected, I click on `Validate`.

.. image:: static/description/eft_validate.png

An error message appears. One of my bank accounts is not properly filled.

.. image:: static/description/eft_validate_error.png

I fix the account number, then I click again on `Validate`.

.. image:: static/description/eft_fixed_validate.png

The EFT is now `Ready`.

.. image:: static/description/eft_ready.png

EFT Approval
------------
The group `Approve EFT` allows to approve the EFT.
This group is intended for the financial director or controller of your company.

.. image:: static/description/eft_approval_group.png

As member of `Approve EFT`, I go to the EFT form view.

.. image:: static/description/eft_list.png
As member of `Approve EFT`, I go to the EFT form view (`Invoicing / Vendors / EFT`).

I verify that the payments are accurate.
If any payment seems odd, I can click on the line and dilldown to the invoices.

Then I click on `Approve`.

.. image:: static/description/eft_approve.png

The `EFT` is now approved.
Then I click on `Approve`. The `EFT` is now approved.

.. image:: static/description/eft_approved.png

Restriction on Canceling Payment
--------------------------------
Once I have payment with EFT linked to it, and the payment is in the following state : `POSTED`, `SENT`, and `RECONCILED`,
I can not cancel the payment anymore. If I still try to cancel it, I get the following message:
Once I have payment with EFT linked to it, and the payment is in `POSTED` state,
I can not reste to draft the payment anymore. If I still try to cancel it, I get the following message:

.. image:: static/description/cancel_payment_usererror.png
.. image:: static/description/reset_to_draft_payment_usererror.png

Generating The File
-------------------
The current step can be done by a member of the group `Accounting / Billing`.
The current step can be done by a member of the group `Invoicing / Billing`.
In the EFT form view, I click on `Generate File`:

.. image:: static/description/eft_generate_file_button.png

Expand All @@ -224,7 +207,7 @@ Otherwise, even if the file was accepted by your bank, some payments may be reje
In such case, the module allows you to identify which payments were successfully transmitted to the recipient account
and which were not.

As member of the group `Accounting / Billing`, I click on `Confirm Sending`.
As member of the group `Invoicing / Billing`, I click on `Confirm Sending`.

.. image:: static/description/eft_confirm_sending.png

Expand All @@ -240,11 +223,10 @@ Multiple changes were applied to the `EFT`.

(1) The EFT is now `Done`.

(2) The 2 succeeding payments are at the status `Sent`.
(2) The 2 succeeding payments are the ckeckbox `Sent` checked.
The payment dates were updated to match the EFT date.

(3) The failed payment is still at the status `Posted`.
This payment can be corrected later and selected into another EFT batch.
(3) The failed payment with checkbox `Sent` Unchecked can be corrected later and selected into another EFT batch.

(4) A new button `Send Payment Notices` appears.
This button allows to notice the recipients by email.
Expand Down
2 changes: 1 addition & 1 deletion canada_bank_transfer/__manifest__.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

{
"name": "Canada Bank Transfer",
"version": "1.2.3",
"version": "1.3.0",
"author": "Savoir-faire Linux,Numigi",
"maintainer": "Numigi",
"website": "https://bit.ly/numigi-com",
Expand Down
Loading

0 comments on commit 85829bd

Please sign in to comment.