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.

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

[16.0][WIP] rma_account_unreconciled: Migration to 16.0 #462

Merged
merged 8 commits into from
Jun 21, 2024

Conversation

AlexPForgeFlow
Copy link
Contributor

@AlexPForgeFlow AlexPForgeFlow commented Oct 9, 2023

Migration to 16.0

(Features pending of new version) This module allows to reconcile those Orders in a single click. In accounting
settings users will be able to set up a specific account for write-off.

ForgeFlow

@AlexPForgeFlow AlexPForgeFlow changed the title [16.0][MIG] rma_account_unreconciled: Migration to 16.0 [16.0][WIP] rma_account_unreconciled: Migration to 16.0 Oct 9, 2023
Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

Works good 👍

CI red just because dependencies not merged.

type="object"
string="Unreconciled Journal Items"
name="action_view_unreconciled"
groups="account.group_account_invoice"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
groups="account.group_account_invoice"
groups="account.group_account_user"

Because standard Odoo does not let access to journal items if not in that group

@AaronHForgeFlow AaronHForgeFlow force-pushed the 16.0-mig-rma_account_unreconciled branch from 2f06a11 to 20f926a Compare October 31, 2023 07:51
ChrisOForgeFlow and others added 7 commits May 9, 2024 14:00
…ompute method fails.

[IMP] rma: use rma user in tests
[FIX] rma_account: move_line_id field string
[IMP] rma, rma_account, rma_sale, rma_purchase: tests for stock valuation
[FIX] account_move_line_rma_order_line: minor lint, make auto-install
[IMP] rma_account_unreconciled: use manager user in test
Otherwise the anglo saxon entries won't be correct.
For example, the Interim (Delivered) account should balance
after receiving and triggering a refund on a customer rma.
@AlexPForgeFlow AlexPForgeFlow force-pushed the 16.0-mig-rma_account_unreconciled branch from 20f926a to d83db3b Compare May 9, 2024 12:03
@BernatPForgeFlow BernatPForgeFlow force-pushed the 16.0-mig-rma_account_unreconciled branch from 25a840a to 27eae39 Compare June 20, 2024 12:10
…ency

RMA lines should be able to reconcile either with OCA or Enterprise reconcile tools.
@BernatPForgeFlow BernatPForgeFlow force-pushed the 16.0-mig-rma_account_unreconciled branch from 27eae39 to def1b22 Compare June 20, 2024 13:41
@JordiBForgeFlow JordiBForgeFlow merged commit b944f47 into 16.0 Jun 21, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants