-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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 TikTok Marketing: fix incremental bug + new reports' streams #8292
Conversation
airbyte-integrations/connectors/source-tiktok-marketing/source_tiktok_marketing/streams.py
Outdated
Show resolved
Hide resolved
/test connector=connectors/source-tiktok-marketing
|
* also use enums for reporting level and granularity
* also add report level and granularity selector to spec.py
/test connector=connectors/source-tiktok-marketing
|
/test connector=connectors/source-tiktok-marketing
|
/test connector=connectors/source-tiktok-marketing
|
995f46f
to
9057a90
Compare
/test connector=connectors/source-tiktok-marketing
|
/test connector=connectors/source-tiktok-marketing
|
/publish connector=connectors/source-tiktok-marketing
|
…airbytehq#8292) * Add BasicReport stream to Tiktok source * Rename config parameter to `start_date` to match specs * Fix TypeError on state comparison with last record's cursor field * Fix date chunks for all granularities * also use enums for reporting level and granularity * Adapt requested metrics based on report level and granularity * also add report level and granularity selector to spec.py * Add schema file for BasicReport stream * update spec file * format * bump version * round reports states * round reports states * fix selection of JsonUpdatedState object * save updated value * fix reports schema * fix data field * test JSON error * update report schema * update report schema * fix tests * update test config * remove test file * test ci failures * debug tests * update test config * update CDK version * fix flake8 comments * update spec file Co-authored-by: n0rritt <dev.n0rritt@gmail.com>
What
correction for #8196
Pre-merge Checklist
Updating a connector
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog exampleAirbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.
/test connector=connectors/<name>
command is passing./publish
command described here