GabrielBBaldez opened a new pull request, #11028: URL: https://github.com/apache/seatunnel/pull/11028
## Purpose of this pull request Adds a **Shopify Source connector** under Connector-V2, as requested in #11027. It follows the existing HTTP-based SaaS source connector pattern (e.g. `Klaviyo`, `Jira`): a thin `connector-http-shopify` module on top of `connector-http-base`. It authenticates with a Shopify Admin API access token sent in the `X-Shopify-Access-Token` header, and reads a Shopify Admin REST endpoint (orders, products, customers, …) into SeaTunnel rows, reusing the base HTTP source's pagination, retry/backoff, JSON-path and schema handling. ### Options - `url` (required) — the Shopify Admin API endpoint, e.g. `https://your-store.myshopify.com/admin/api/2024-01/products.json` - `access_token` (required) — sent as the `X-Shopify-Access-Token` header - plus the common HTTP source options (`method`, `schema`, `format`, `params`, `json_field`, `content_json`, retry/backoff, …) ### Scope This is a focused V1 aligned with the current SeaTunnel HTTP-connector conventions (per the issue: *"start as a focused batch source connector"*). Richer capabilities mentioned in the issue — a `resources` list, Shopify cursor-based pagination, and incremental extraction — can follow in a later iteration. Happy to align option names and scope during review. ### Added - `seatunnel-connectors-v2/connector-http/connector-http-shopify` module (`ShopifySource`, `ShopifySourceFactory`, `ShopifySourceOptions`, `ShopifySourceParameter`) + factory test - Registered in `connector-http/pom.xml`, `plugin-mapping.properties`, and `seatunnel-dist/pom.xml` - English and Chinese docs: `docs/{en,zh}/connectors/source/Shopify.md` ## Does this PR introduce _any_ user-facing change? Yes — adds a new `Shopify` source connector. ## How was this patch tested? Added a factory option-rule test (same as the other HTTP source connectors) and verified code style with `spotless`. The full multi-module build/IT runs in CI. ## Check list - [x] Code changed are covered with tests, or it does not need tests for reason - [x] If any new Jar binary package adding in your PR, please add License Notice according [New License Guide](https://github.com/apache/seatunnel/blob/dev/docs/en/contribution/new-license.md) - [x] If necessary, please update the documentation to describe the new feature. Closes #11027 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
