Skip to content
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

source-apple-search-ads contribution from aalkuatova #55785

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aalkuatova
Copy link

What

This PR updates source Apple Search Ads (source-apple-search-ads).

The contributor provided the following description of the change:

Connector for Apple Search Ads.

Reviewer checklist

  • Resolve any merge conflicts and validate file diffs (make sure the PR only includes changes intended by the contributor)
  • After reviewing the changes, run the bump-version Airbyte-CI command locally to update the version of the connector according to the versioning guidelines. Add breakingChanges to metadata if necessary.
  • Ensure connector docs are up to date with any changes
  • Run /format-fix to resolve any formatting errors
  • Click into the CI workflows that wait for a maintainer to run them, which should trigger CI runs

Copy link

vercel bot commented Mar 16, 2025

@aalkuatova is attempting to deploy a commit to the Airbyte Growth Team on Vercel.

A member of the Team first needs to authorize it.

@aalkuatova
Copy link
Author

@marcosmarxm I did use the latest version 6.36.3 but then had the following error: "Your YAML manifest is not compatible with the Connector Builder UI, due to the following error: Error handling request: The manifest version 6.36.3 is greater than the airbyte-cdk package version (6.33.4). Your manifest may contain features that are not in the current CDK version. There is no UI value to switch back to. Please resolve this error with the YAML editor in order to switch to UI mode". So I just changed the manifest version to 6.33.4 as kapa.ai suggested cause I have the latest abctl version - v0.24.0. Added the timezone parameter where it was needed and tested all streams.

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

Successfully merging this pull request may close these issues.

2 participants