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

[14.0] FIX account.invoice.report considering receipts also #115643

Closed
wants to merge 1 commit into from

Conversation

eLBati
Copy link
Contributor

@eLBati eLBati commented Mar 17, 2023

Description of the issue/feature this PR addresses:

Allow to analyze receipts

Current behavior before PR:

  • User registers receipts
  • User is not able to analyze data about them

Desired behavior after PR is merged:

  • User is able to analyze receipts data
  • Default behavior of "Invoices Analysis" is based on invoices
  • User can select filters to show receipts data

We could add a specific report for receipts, but it is useful to have aggregate data


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo
Copy link
Contributor

robodoo commented Mar 17, 2023

@C3POdoo C3POdoo requested review from a team and qdp-odoo and removed request for a team March 17, 2023 09:57
@lordkrandel lordkrandel self-requested a review March 31, 2023 06:08
@eLBati
Copy link
Contributor Author

eLBati commented Mar 31, 2023

Including in OCA/account-invoicing#1427

@lordkrandel
Copy link
Contributor

Looks like the partner's not interested anymore in continuing this, but the PR is good.
@jco-odoo is this something we want? Should we make a PR/task for 15.0 (or later)?

@lordkrandel
Copy link
Contributor

I'm closing this since we don't support Odoo 14.0 anymore.
Looks like neither @qdp-odoo or @jco-odoo are interested.
For feature requests like this, please target at least Odoo 17 or master, or ask account and partner managers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants