[18.0][ADD] shoppingfeed_integration#1
Open
juancarlosonate-tecnativa wants to merge 27 commits into
Open
Conversation
juancarlosonate-tecnativa
force-pushed
the
18.0-add-shoppingfeed_integration
branch
4 times, most recently
from
5c92337 to
ea1328e
Compare
Author
|
Could you take a look when you have some time, please? @christian-ramos-tecnativa @carlos-lopez-tecnativa @eduezerouali-tecnativa |
christian-ramos-tecnativa
suggested changes
Dec 1, 2025
christian-ramos-tecnativa
left a comment
christian-ramos-tecnativa
left a comment
There was a problem hiding this comment.
Excellent work! It's a major module, so I haven't finished testing it yet, but there are a couple of aspects that could be improved
| "will be exported with quantity 0." | ||
| ), | ||
| ) | ||
| update_quantities_realtime = fields.Boolean( |
There was a problem hiding this comment.
In the stock related fields I was missing a warehouse filtering. So I could for example specify different warehouses for different shoppingfeed stores. WDYT
| ] | ||
| ) | ||
| ) | ||
| for product in self.filtered("default_code"): |
There was a problem hiding this comment.
You are always taking the product default_code as reference, but shouldn't you check the reference used in the store? The fields use_product_id_as_sku and custom_sku_field_id at shoppingfeed.store model
juancarlosonate-tecnativa
force-pushed
the
18.0-add-shoppingfeed_integration
branch
4 times, most recently
from
f4cc37c to
f7cb778
Compare
juancarlosonate-tecnativa
force-pushed
the
18.0-add-shoppingfeed_integration
branch
from
f7cb778 to
4fabc67
Compare
juancarlosonate-tecnativa
force-pushed
the
18.0-add-shoppingfeed_integration
branch
3 times, most recently
from
b0394aa to
9dfdfea
Compare
carlos-lopez-tecnativa
suggested changes
Jan 16, 2026
juancarlosonate-tecnativa
force-pushed
the
18.0-add-shoppingfeed_integration
branch
2 times, most recently
from
8178d9b to
2aa1193
Compare
carlos-lopez-tecnativa
suggested changes
Jan 20, 2026
juancarlosonate-tecnativa
force-pushed
the
18.0-add-shoppingfeed_integration
branch
6 times, most recently
from
e3aa9a1 to
d2eb756
Compare
juancarlosonate-tecnativa
force-pushed
the
18.0-add-shoppingfeed_integration
branch
4 times, most recently
from
0c3de7d to
fc1f3bd
Compare
sergio-teruel
force-pushed
the
18.0-add-shoppingfeed_integration
branch
from
d1aed44 to
70a5508
Compare
When this feature is enabled (by default), new orders are confirmed automatically.
…ed for orders imported from the API
…account to retrieve sale orders from channel TT62541
carlos-lopez-tecnativa
suggested changes
May 19, 2026
sergio-teruel
force-pushed
the
18.0-add-shoppingfeed_integration
branch
from
1691d60 to
5f54dcf
Compare
…ale orde line TT62572
…e to many2many in store carrier mappings With this change, for each mapping line the user can configure multiple carriers, and the logic selects the first one that matches the shipping address. For example, the customer buys from the Spain store, but the delivery address is in another country.
…shipping addresses
REMOVE BEFORE MERGE
carlosdauden
force-pushed
the
18.0-add-shoppingfeed_integration
branch
2 times, most recently
from
44e22b8 to
38edbfb
Compare
Method _shoppingfeed_format_phone called without order and _phone_format has ensure_one TT62646
carlosdauden
force-pushed
the
18.0-add-shoppingfeed_integration
branch
from
38edbfb to
680bc80
Compare
… number is set Before this commit, the formatting was never performed because self was always empty. In this commit: OCA@44e22b8 the code was changed to skip formatting when self is empty, but self is always empty in this case. This commit fixes the code to perform the formatting only when the phone field is set; if it is empty, the formatting is skipped. https://github.com/odoo/odoo/blob/74995277d6817103ae1816e4653e81460c3daf56/addons/phone_validation/models/models.py#L74
juancarlosonate-tecnativa
force-pushed
the
18.0-add-shoppingfeed_integration
branch
from
8cf9af1 to
1b3f972
Compare
…ch orders since a given datetime TT62769
juancarlosonate-tecnativa
force-pushed
the
18.0-add-shoppingfeed_integration
branch
2 times, most recently
from
1b3f972 to
2907d8b
Compare
…in delivery partners TT62772
… child partner creation TT62772
…d duplicate contacts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@Tecnativa TT56368
@carlosdauden @sergio-teruel