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
[ADD] pharmacy_retail: Pharmacy Retail #45
base: 17.0
Are you sure you want to change the base?
Conversation
079d447
to
ccdd706
Compare
b10cbae
to
d0b0eaf
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.
Here are some basic improvements!
Please apply them several times when needed.
Thank you
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.
Remove
@@ -0,0 +1,78 @@ | |||
# -*- coding: utf-8 -*- |
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.
# -*- coding: utf-8 -*- |
""", | ||
'author': 'Odoo S.A', | ||
'depends': [ | ||
'account_check_printing', |
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.
'account_check_printing', |
'purchase_stock', | ||
'sale_loyalty', | ||
'sale_margin', | ||
'sale_planning', |
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.
'sale_planning', |
Remove unused dependency
<field name="name">Himalaya Baby Cream - Extra Soft & Gentle, Paraben Free, 100 ml Tube</field> | ||
<field name="sequence">14</field> | ||
<field name="order_id" ref="pharmaceuticals_retailers.sale_order_3" /> | ||
<field name="order_partner_id" ref="pharmaceuticals_retailers.res_partner_22" /> |
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="order_partner_id" ref="pharmaceuticals_retailers.res_partner_22" /> |
<field name="order_id" ref="pharmaceuticals_retailers.sale_order_3" /> | ||
<field name="order_partner_id" ref="pharmaceuticals_retailers.res_partner_22" /> | ||
<field name="product_id" ref="pharmaceuticals_retailers.product_product_20" /> | ||
<field name="product_uom" ref="uom.product_uom_unit" /> |
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="product_uom" ref="uom.product_uom_unit" /> |
<field name="product_id" ref="pharmaceuticals_retailers.product_product_20" /> | ||
<field name="product_uom" ref="uom.product_uom_unit" /> | ||
<field name="product_uom_qty">3.0</field> | ||
<field name="qty_delivered_method">stock_move</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="qty_delivered_method">stock_move</field> |
<record id="stock_lot_20" model="stock.lot"> | ||
<field name="name">A-003</field> | ||
<field name="product_id" ref="pharmaceuticals_retailers.product_product_19" /> | ||
<field name="product_uom_id" ref="pharmaceuticals_retailers.uom_uom_27" /> | ||
</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.
Remove unused lots
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.
Remove unused images
b7cf96e
to
0aa42e6
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.
Remove
This setup is for pharmacy retail companies that purchase and sell products. | ||
They maintain the stock lot wise and also manage the expiry of the lots. | ||
""", | ||
'author': 'Odoo S.A', |
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.
'author': 'Odoo S.A', |
'demo/sale_order_line.xml', | ||
'demo/sale_order_post.xml', | ||
], | ||
'application': False, |
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.
'application': False, |
@@ -0,0 +1,113 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> | |||
<odoo> | |||
<record id="doctor_34fd41e9-4456-47f4-bb18-88a9886e843e" model="ir.actions.act_window"> |
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.
Remove hash
@@ -0,0 +1,71 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> | |||
<odoo> | |||
<record id="generic_name_group_s_b0925917-99ee-4fbe-b5d5-05a4f48a825b" model="ir.model.access"> |
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.
remove hash
<field name="partner_id" ref="pharmaceuticals_retailers.res_partner_18" /> | ||
</record> | ||
<record id="purchase_order_5" model="purchase.order"> | ||
<field name="partner_id" ref="pharmaceuticals_retailers.res_partner_19" /> |
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="partner_id" ref="pharmaceuticals_retailers.res_partner_19" /> | |
<field name="partner_id" ref="res_partner_19" /> |
<record id="res_partner_24" model="res.partner"> | ||
<field name="name">Marc Demo</field> | ||
</record> | ||
<record id="res_partner_25" model="res.partner"> | ||
<field name="name">Barak</field> | ||
</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.
<record id="res_partner_24" model="res.partner"> | |
<field name="name">Marc Demo</field> | |
</record> | |
<record id="res_partner_25" model="res.partner"> | |
<field name="name">Barak</field> | |
</record> |
remove if unused
<field name="order_id" ref="pharmaceuticals_retailers.sale_order_3" /> | ||
<field name="product_id" ref="pharmaceuticals_retailers.product_product_20" /> | ||
<field name="product_uom_qty">3.0</field> | ||
<field name="price_unit">148.8</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="price_unit">148.8</field> |
<odoo noupdate="1"> | ||
<record id="stock_lot_1" model="stock.lot"> | ||
<field name="name">A-001</field> | ||
<field name="product_id" ref="pharmaceuticals_retailers.product_product_13" /> |
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="product_id" ref="pharmaceuticals_retailers.product_product_13" /> | |
<field name="product_id" ref="product_product_13" /> |
<record id="stock_lot_1" model="stock.lot"> | ||
<field name="name">A-001</field> | ||
<field name="product_id" ref="pharmaceuticals_retailers.product_product_13" /> | ||
<field name="product_uom_id" ref="pharmaceuticals_retailers.uom_uom_32" /> |
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="product_uom_id" ref="pharmaceuticals_retailers.uom_uom_32" /> |
related field
c4124d7
to
6e5706b
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.
Remove
<field name="type">form</field> | ||
<field name="arch" type="xml"> | ||
<form> | ||
<header /> |
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.
<header /> |
<field name="type">form</field> | ||
<field name="arch" type="xml"> | ||
<form> | ||
<header /> |
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.
<header /> |
<header /> | ||
<sheet string="Therapeutic Classification"> | ||
<widget name="web_ribbon" text="Archived" bg_color="text-bg-danger" /> | ||
<field name="x_active" invisible="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.
<field name="x_active" invisible="1" /> |
<field name="margin">5.8</field> | ||
<field name="pricelist_id" ref="product_pricelist_3"/> | ||
<field name="x_doctor" ref="res_partner_10"/> | ||
<field name="amount_to_invoice">64.17</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="amount_to_invoice">64.17</field> |
<record id="sale_order_line_1" model="sale.order.line" context="{'skip_task_generation': True, 'skip_purchase_generation': True}"> | ||
<field name="order_id" ref="sale_order_1" /> | ||
<field name="product_id" ref="product_product_13" /> | ||
<field name="product_uom" ref="uom_uom_32" /> |
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="product_uom" ref="uom_uom_32" /> |
comes from the product
<field name="order_id" ref="sale_order_4" /> | ||
<field name="product_id" ref="product_product_13" /> | ||
<field name="product_uom" ref="uom_uom_32" /> | ||
<field name="qty_to_invoice">1.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.
<field name="qty_to_invoice">1.0</field> |
<record id="sale_order_line_15" model="sale.order.line" context="{'skip_task_generation': True, 'skip_purchase_generation': True}"> | ||
<field name="order_id" ref="sale_order_3" /> | ||
<field name="product_id" ref="product_product_26" /> | ||
<field name="qty_delivered_method">manual</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="qty_delivered_method">manual</field> |
<record id="stock_lot_1" model="stock.lot"> | ||
<field name="name">A-001</field> | ||
<field name="product_id" ref="product_product_13" /> | ||
</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.
<record id="stock_lot_1" model="stock.lot"> | |
<field name="name">A-001</field> | |
<field name="product_id" ref="product_product_13" /> | |
</record> |
Remove unused lots
532dce4
to
938628b
Compare
Hello @paso-odoo Have a great day ahead 😇 |
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.
First very quick functional review
938628b
to
54a8f5e
Compare
Hello @vava-odoo Thanks for your suggestion and have a great day ahead 😇 |
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.
Hi @shag-odoo
Thanks for the update. I listed a few other comments, before going deeper into the data.
What I don't understand is why we need new ir.model here... It seems to me none should be added.
Could you have a look?
Thanks!
54a8f5e
to
e54af2f
Compare
d71a80d
to
f45aad7
Compare
cfa4efc
to
491291e
Compare
491291e
to
2450b0a
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.
Thanks @arpa-odoo for the update!
a1430b9
to
6784e1a
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.
Hi @arpa-odoo
Just a few comments here, and I think it's good to go.
Cheers
<field name="payment_method_ids" eval="[ | ||
Command.link(ref('payment.payment_method_card')), | ||
Command.link(ref('payment.payment_method_visa')), | ||
Command.link(ref('payment.payment_method_mastercard')), | ||
Command.link(ref('payment.payment_method_rupay')), | ||
Command.link(ref('payment.payment_method_maestro')), | ||
]"/> |
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.
Isn't it the default?
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.
no
…mo data Added demo data for pharmaceuticals retailers undustry task-3460554
f616114
to
d74fbfd
Compare
d74fbfd
to
70187cc
Compare
Task-3460554