-
-
Notifications
You must be signed in to change notification settings - Fork 182
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][IMP] sale_stock_available_to_promise_release add ir.config_pa… #896
base: 16.0
Are you sure you want to change the base?
[16.0][IMP] sale_stock_available_to_promise_release add ir.config_pa… #896
Conversation
75b8eb0
to
ef41a00
Compare
sale_stock_available_to_promise_release/models/res_config_settings.py
Outdated
Show resolved
Hide resolved
8ec116f
to
25565b2
Compare
sale_stock_available_to_promise_release/models/res_config_settings.py
Outdated
Show resolved
Hide resolved
25565b2
to
ef12db2
Compare
ef12db2
to
fbf0c70
Compare
sale_stock_available_to_promise_release/models/res_config_settings.py
Outdated
Show resolved
Hide resolved
sale_stock_available_to_promise_release/views/res_config_settings.xml
Outdated
Show resolved
Hide resolved
db179ca
to
75688f1
Compare
75688f1
to
cb6d778
Compare
cb6d778
to
7734638
Compare
This PR has the |
|
||
stock_sale_report_display_is_available = fields.Boolean( | ||
help="Show Available column in SO and quotation reports", | ||
default=True, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be False
by default ?
cc @sebalix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Before this PR the column was always displayed, we are only adding an option here to hide it, but by default we keep it displayed as before. No?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all good then :)
@victorvermot Can you drop |
Add parameter to make the column Available on SO reports invisible
Results