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.

Sign up for GitHub

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

Jump to bottom

While import sale order, correct product is not set inside shipping line according to carrier method. #112

Closed
akshay-emipro opened this issue Jul 3, 2015 · 6 comments
Closed

While import sale order, correct product is not set inside shipping line according to carrier method. #112

akshay-emipro opened this issue Jul 3, 2015 · 6 comments
Assignees
@guewen

Comments

@akshay-emipro
Copy link

While import sale order from Magento to Odoo, always default product "[SHIP]" is set inside shipping line of that order. Even though we have specified product inside carrier method of that order.

System is not going to getting correct product from selected carrier method of that order instead of system always taking default service product which have external id "connector_ecommerce.product_product_shipping".

We have been tested with Odoo V8. Also getting it inside OpenERP V7.
@akshay-emipro akshay-emipro mentioned this issue Jul 3, 2015
Merged
guewen added a commit to guewen/connector-magento that referenced this issue Jul 3, 2015
@guewen
Wrong attribute name for the shipping line builder
547d624 
Fixes OCA#112
guewen added a commit to guewen/connector-magento that referenced this issue Jul 3, 2015
@guewen
Wrong attribute name for the shipping line builder
290d05c 
Fixes OCA#112
@guewen guewen mentioned this issue Jul 3, 2015
Closed
@guewen guewen self-assigned this Jul 3, 2015
guewen added a commit to guewen/connector-magento that referenced this issue Jul 3, 2015
@guewen
Wrong attribute name for the shipping line builder
508f0c5 
Fixes OCA#112
@guewen guewen mentioned this issue Jul 3, 2015
Open
@guewen
Copy link
Member

guewen commented Jul 3, 2015

Could you test this patch #114 and report back you result on the pull request?

@akshay-emipro
Copy link
Author

I have report back with your given changes. Please have a look on it. Thanks :)

@guewen
Copy link
Member

guewen commented Jul 3, 2015

Sorry, I didn't saw your pull request that's why I proposed another one 🙈

@guewen
Copy link
Member

guewen commented Jul 8, 2015

@emiprotechnologies could you have the information you want? Are you going to move forwards with the CLA?

@akshay-emipro
Copy link
Author

I have signed CLA. Where can I see list of CLA members ? Where can I see my own details ?
Thanks.

@elicoidal
Copy link

I have a ICLA on Chirag Rajkotiya for this github in our DB. I should be good for the bot
If you have more msg for clabot, contact me

@guewen guewen closed this as completed in 713f31f Jul 30, 2015
jcoux pushed a commit to camptocamp/connector-magento that referenced this issue May 14, 2019
@CharlineDumontet
Merge pull request OCA#112 from camptocamp/1.2.17
2e3df42 
1.2.17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@guewen guewen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@guewen @elicoidal @akshay-emipro