source-woocommerce: ensure inline schemas, updated cdk, poetry, generate builder manifest #37289
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
octavia-squidington-iii
added
area/connectors
bleonard
changed the title
airbyte-integrations/connectors/source-woocommerce/source_woocommerce/schemas/coupons.json
Outdated
Show resolved
Hide resolved
strosek
changed the title
| | 0.2.0 | 2022-11-30 | [19903](https://github.com/airbytehq/airbyte/pull/19903) | Migrate to low-code; Certification to Beta | | ||
| | 0.1.1 | 2021-11-08 | [7499](https://github.com/airbytehq/airbyte/pull/7499) | Remove base-python dependencies | | ||
| | 0.1.0 | 2021-09-09 | [5955](https://github.com/airbytehq/airbyte/pull/5955) | Initial Release. Source WooCommerce | | ||
| | 0.2.7 | 2024-05-14 | [37289](https://github.com/airbytehq/airbyte/pull/37289) | Updating to 0.88.0 CDK | |
There was a problem hiding this comment.
Have we published previous versions of the connector related to this PR?
There was a problem hiding this comment.
I don't think so. But the number changes came from the automated scripts run by Brian.
|
|
||
| [tool.poetry.dependencies] | ||
| python = "^3.9,<3.12" | ||
| airbyte-cdk = "0.88" |
There was a problem hiding this comment.
Why do we hardcode from the minor version?
There was a problem hiding this comment.
@maxi297 told me it was better to keep the connector stable in case of future changes.
| schema = stream.json_schema | ||
| if not schema: |
There was a problem hiding this comment.
It would be better to move those changes to another PR.
There was a problem hiding this comment.
I added this change so the PR can run clean CATs, othrewise CI wouldn't pass. Is it OK to keep them?
fix typo in Readme Co-authored-by: Serhii Lazebnyi <53845333+lazebnyi@users.noreply.github.com>
|
Closed to avoid conflicts with Brian's work. |
This was created from a set of automated scripts. In each case, not every update was needed for every connector, but overall here is what happened:
auto-schema update -s all description: added descriptions to json and inline schemasconnector-code migrate_to_yaml -c all --type source: migrates json schemas to connectors with a manifestairbyte-ci connectors --name=<all modified connectors from above> migrate_to_base_image: makes sure that each is using a base docker image and updates docsairbyte-ci connectors --name=<all modified connectors from above> migrate-to-poetry: moves connectors not already using poetry to do so and updates documentationairbyte-ci connectors --name=<all modified connectors from above> up_to_date: updated the CDK to newer (0.88.0) versionThe version number and changelogs were also bumped using the
connector-code pullscommand.Additionally a few changes were made on top of the above changes: