Skip to content

draft: [16.0][ADD] connector_amazon_spapi 2#513

Open
kobros-tech wants to merge 30 commits intoOCA:16.0from
kencove:add-connector_amazon_spapi-2
Open

draft: [16.0][ADD] connector_amazon_spapi 2#513
kobros-tech wants to merge 30 commits intoOCA:16.0from
kencove:add-connector_amazon_spapi-2

Conversation

@kobros-tech
Copy link

using python-amazon-sp-api

dnplkndll and others added 30 commits December 19, 2025 00:39
The @job decorator from queue_job is not needed when using
the with_delay() pattern. Removed the import and decorator
usage to fix ImportError in CI environment.
The view uses mail chatter with message_follower_ids and message_ids
fields, which require mail.thread and mail.activity.mixin. Adding
these mixins enables the mail chatter functionality in the form view.
The amazon.feed model has a 'state' field, not 'status'.
Updated both tree and form views to use the correct field name.
- test_feed.py: Complete feed submission workflow tests
  - Happy path: 4-step submission (document, upload, feed, status)
  - Error handling: API errors, S3 upload failures
  - Status checking: IN_PROGRESS, DONE, FATAL, CANCELLED
  - Retry logic and multiple feed types

- test_shop.py: Stock push and competitive pricing tests
  - push_stock() creates and submits inventory feeds
  - _build_inventory_feed_xml() generates valid XML structure
  - Negative stock handling (sends 0, not negative)
  - cron_push_stock() processes hourly/daily shops
  - cron_push_shipments() queues shipment jobs
  - Timestamp updates for sync operations

Refs: COVERAGE_IMPROVEMENT_PLAN.md items 4-5
…ns in order.py

- Remove duplicate _get_last_done_picking at line 53 (simpler filter-based version)
- Remove duplicate push_shipment at line 63 (basic implementation version)
- Keep robust implementations with better error handling and test compatibility
- Fixes flake8 F811 redefinition errors blocking pre-commit validation
@kobros-tech
Copy link
Author

@jelenapoblet
@clementmbr

here is an update using third party connector

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants