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

[7.0] Add module 'customer_credit_limit' #9

Closed
wants to merge 6 commits into from

Conversation

JordiBForgeFlow
Copy link
Sponsor Member

I have reviewed modules 'customer_credit_limit' by Sistemas ADHOC and ECOSOFT and incorporated:

  • A group that can approve a quotation, even when the credit limit is exceeded
  • The possibility to manually run the credit limit check before the quote is attempted to be approved. This makes it possible for the sales user to run the credit limit check before printing and sending the quotation to the customer for approval.

+ credit limit defined in the customer master
- Partner's credit (considering total open AR / AP items)
- Open order lines (not invoiced)
- Open customer invoices
+ Open customer refunds
in status draft or sent.

When approving a Sale Order it computes the sum of:
* The Accounts Receivable Journal Items pending to be pai
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

to be paid


<record id="group_so_credit_block_releaser" model="res.groups">
<field name="name">Sales Order Credit Block Releaser</field>
<field name="category_id" ref="base.module_category_sales_management"/>
Copy link
Member

Choose a reason for hiding this comment

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

This will break the rendering of the original sales groups in the dropdown. Please give it another category, or (my personal preference) assign the privileges to the sales manager instead of adding a dedicated group for this.

@StefanRijnhart
Copy link
Member

cc original authors @jjscarafia, @kittiu (I believe)

@StefanRijnhart
Copy link
Member

You have some flake8 to resolve, and a test fails. Maybe you can set the default limit sufficiently high to make the tests succeed?

@StefanRijnhart StefanRijnhart added this to the 7.0 milestone May 18, 2016
@StefanRijnhart
Copy link
Member

Ow, I see this PR is from Dec 2014, not Dec 2015 as I thought ;-(

@pedrobaeza
Copy link
Member

Closing as stated in #292 as better approach.

@pedrobaeza pedrobaeza closed this Oct 10, 2016
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