-
Notifications
You must be signed in to change notification settings - Fork 83
[IMP] agriculture_shop: Remove survey, lot and add variants, discount #1228
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
base: 19.0
Are you sure you want to change the base?
Conversation
a3ab6a7 to
cc6c688
Compare
cc6c688 to
68b1a42
Compare
New version of the agriculture shop module: - Remove the survey app and the lots and SN setting. - Add variant and discount settings - Add ecommerce categories - Add quantity and variants on products - Change picture for product "Battery spray pump" TASK-5123814
68b1a42 to
53594e5
Compare
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.
Please also update the pot (test was removed, but we still need to)
| @@ -1,14 +1,14 @@ | |||
| { | |||
| 'name': 'Agricultural Store', | |||
| 'version': '1.1', | |||
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.
| 'version': '1.1', | |
| 'version': '1.2', |
| the <b>Suggest toggle</b> on the catalog (available from the purchase line).</p> | ||
| <p> | ||
| <img | ||
| src="https://190-agrishop.odoo.com/web/image/1401-0d2effae/image.png?access_token=d7b4f3f8-4987-4678-88d2-e867dbc9487a" |
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.
😮
| </record> | ||
| <record id="product_attribute_value_8" model="product.attribute.value"> | ||
| <field name="name">0.5 Kg</field> | ||
| <field name="sequence">1</field> |
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.
better use auto_sequence, no?
| <field name="sequence">1</field> |
| <odoo noupdate="1"> | ||
| <record id="product_image_1" model="product.image"> | ||
| <field name="image_1920" type="base64" file="agriculture_shop/static/src/binary/product_image/1-image_1920"/> | ||
| <field name="name">SPray.webp</field> |
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.
in the backend, it should be a jpg/png, not a webp (the website will do the automatic conversion)
| <field name="expiration_time">10</field> | ||
| <field name="available_in_pos" eval="True"/> | ||
| <field name="pos_categ_ids" eval="[(6, 0, [ref('pos_category_4')])]"/> | ||
| <field name="qty_available">20.0</field> |
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.
Didn't know we could set the quantity available that way. But since it creates stock.quant, this should be done in demo/
| <field name="street">13619 Dewitt Rd, Dewitt, MI 48820, USA</field> | ||
| <field name="zip">48820</field> | ||
| <field name="city">DeWitt</field> | ||
| <field name="reveal_id">52de6119-6228-40ea-abaa-35ee4ed09e07</field> |
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.
| <field name="reveal_id">52de6119-6228-40ea-abaa-35ee4ed09e07</field> |
| <record id="sale_order_line_12" model="sale.order.line"> | ||
| <field name="order_id" ref="sale_order_6"/> | ||
| <field name="product_id" ref="product_product_34"/> | ||
| <field name="price_unit">-1400.0</field> | ||
| </record> | ||
| <record id="sale_order_line_11" model="sale.order.line"> | ||
| <field name="order_id" ref="sale_order_6"/> | ||
| <field name="sequence">13</field> | ||
| <field name="display_type">line_note</field> | ||
| <field name="name"><![CDATA[This quotation receives the current discount. A 45% discount will be applied to your final qualifying order if you purchase a total volume equal to ten times this quotation's amount within the next year.]]></field> | ||
| <field name="product_uom_qty" eval="False"/> | ||
| </record> |
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.
A discount should be added via a function call
| </record> | ||
| <record id="sale_order_line_13" model="sale.order.line"> | ||
| <field name="order_id" ref="sale_order_6"/> | ||
| <field name="product_id" ref="delivery.product_product_delivery"/> |
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.
same for delivery

New version of the agriculture shop module:
TASK-5123814