-
-
Notifications
You must be signed in to change notification settings - Fork 170
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
[MIG] connector_prestashop: Migration to 15.0 #179
Open
Neitherkx
wants to merge
101
commits into
OCA:15.0
Choose a base branch
from
admingrupoisonores:pr-mig-connector
base: 15.0
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains 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
Neitherkx
force-pushed
the
pr-mig-connector
branch
from
May 27, 2024 10:02
e1052c8
to
036340f
Compare
* [FIX] manufacturer export on template Export of manufacturer on template was broken due to a probable typo. `id_manufacturer` value was taken using `to_odoo` instead of `to_backend`. * [IMP] filter of importable orders by state If `backend_record.importable_order_state_ids` is valued we check if current order is in the list. If not, the job fails gracefully. * [FIX] combination image: do not default to `False` when missing
* [FIX] Fix export when no location has flag prestashop_synchronized When there is no location with the flag prestashop_synchronized in the location tree, Current behavior: The `location` key in the context is empty then the qty_available is computed for *every* locations/warehouses. Expected behavior: If there is at least one location flagged, use it, otherwise, use all the locations of the tree, starting from the selected location or selected warehouse location. When no location can be found (for instance because user selected a location with a usage different than 'internal'), then it should just fail and not return the stock of all warehouses. Being able to synchronize without activating 'prestashop_synchronized' is important: when used, this feature will trigger an export every time a quant is changed or created, on a large catalog, we might prefer only using the cron to have less frequent updates and less jobs. A second fix is that when a stock_location_id is set on the backend, it should be used, it was only used for the import of stock. * [FIX] Exclude children from qty computation We already pass the children in 'location', if we don't disable 'compute_child', the qty computation will try to get the children of each child and generates query of 1mio chars. * [IMP] Group computation of qty_available Stats with 6700 products: Before: 570s After: 28s * [FIX] Apply filter on locations for variants quantities It was only applied on export of template quantities. * [IMP] Improve performance on prestashop qty recompute The add/union operation between recordsets is slow, aggregate the ids in a set and call the browse only once on the complete set is dramatically faster. See guewen/connector-magento#9
That may pass the prestashop id insted of the binding id to the exporter That can cause the following problems: * A job Error: Key (id_attribute_group)=(4) is not present in table "prestashop_product_combination_option" * Wrong combination or combination values export or updated * Combination or combination value never exported * Faulty value in fields 'id_attribute_group' leading to data incoherence (cherry picked from commit 767ba36)
* migrate components * port backend buttons remove GenericAdapter * remove unit_for fix key version * start fixing order import * Component usage * [FIX] product views * [FIX] remove prestashop decorator and various fixes
* Wrong imports * Add debug Options * Make mammping more generic * Use new Component for importers * Reference the Components for lates OCA/connector * Fix python import * [REF]Change the way shop_group are referenced * [IMP] Add debug options + requirements (OCA#85) * [REF]Rename views as the model name * [REF]Migrate new components structure
* merge * component usage * product views * remove prestshop decorator and various fixes
* Prevent unicode to be used as filter. particulary tru for the first import * Add menus for finding intermediate object easily * Better way to handle direct matching with components * Add Fields for more options Redefine the Checkpoint method * Add dependencies for stock fields * Clean the add_checkpoint part * Better Views and easier menus * Error lines where not appended but pushed into a list * Prevent duplication of useless delivery carrier * Add options in stock field
* FIX duplicate key * FIX bad adpter for testing connexion * fix bad field for binding * Implement the product matching functionnality * Add active button * Better defensive method on matching delivery carriers * more logs * Better sql constraint * Prevent jobs to be executed on inactive backends * Preserve unique mapping on delivery methods
* [IMP] migrate export of inventory, sale state and tracking number * [IMP] fixes and add no_export field * [FIX] listener * [FIX] temprorary remove unfinished fonctionality * [IMP] continue migration * [FIX] various fixes * [FIX] stock export
* Do not import product template stock into product variants * Update combination quantity when variant stock changes * Set variant images also for default combination * Import supplier prices * Do not duplicate payment modes for each language * connector_prestashop: Do not set default product taxes if they are unset in Prestashop
Due to the total mess on commit history, I finally ended resolving merge conflicts with automated tools and this is the final diff for letting things as they were.
* 🍏 flake8 * 🍏 pylint * 🍏 tests * ©️ Add me as contributor
* New versions of vcrpy has breaking changes, so we freeze it * Adapt .travis.yml for proper runbot deployment * Remove required on account.tax.group.company_id
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: connector-prestashop-10.0/connector-prestashop-10.0-connector_prestashop Translate-URL: https://translation.odoo-community.org/projects/connector-prestashop-10-0/connector-prestashop-10-0-connector_prestashop/
Due to commit mangling, solving conflicts taking incoming change produces some desync that is fixed here in this extra commit, although the attribution for these changes is not correct, but it's the only way to get the exact content.
…tions that don't exist anymore
…refuse order value by default
* This is a combination of 2 commits. [IMP] catalog_manager: New PrestaShop fields for low stock Conflicts: connector_prestashop_catalog_manager/models/product_template/exporter.py [FIX] catalog_manager: Fix export of variants stock from product
Prestashop field "visibility"
* This is a combination of 2 commits. [IMP] connector_prestashop: New keys for version 1.6.1.12 [IMP] connector_prestashop: New keys for version 1.6.1.6
* This is a combination of 4 commits. [WIP] connector_prestashop_catalog_manager: Migration to 12.0 [MIG] connector_prestashop_catalog_manager: Migration to 12.0 [FIX] connector_prestashop_catalog_manager: fixed travis errors. [FIX]connector_prestashop_catalog_manager: Added dependency to product_multi_image.
prestashop.product.template view shows field name and field value.
…restahop. * This is a combination of 3 commits. [FIX] connector_prestashop: Fixed code error on Import Refunds from Prestahop. [FIX]connector_prestashop: Scheduler import payment methods must call import_payment_modes [FIX] connector_prestashop: Set float precision on mapper.
crashed when importing sale.order with unmapped product combinations.
product image update exportation.
unique combination.
* This is a combination of 7 commits. [IMP] connector_prestashop: black, isort, prettie, python3 super syntax [FIX,IMP] odoo 14.0 changes, remove two duplicated records [IMP] connector_prestashop: change multi image dependency, have name in form [FIX] connector_prestashop: missing attributes, local field name [FIX] connector_prestashop: run all test [IMP] connector_prestashop: black, isort, prettier [IMP] connector_prestashop: backend default language, fix import message, assert tax of tax group, show queue job start, match address type
Neitherkx
force-pushed
the
pr-mig-connector
branch
from
August 19, 2024 09:08
e3d080e
to
3370715
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This is a supersed from #151