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 Facebook Marketing - add action report time enum to adinsights class #26345
Conversation
…ights stream FBMarketing
…ebook_marketing/add_action_report_time_enum_to_adinsights_class
…ebook_marketing/add_action_report_time_enum_to_adinsights_class
…ebook_marketing/add_action_report_time_enum_to_adinsights_class
…ebook_marketing/add_action_report_time_enum_to_adinsights_class
…ebook_marketing/add_action_report_time_enum_to_adinsights_class
…t_time_enum_to_adinsights_class
…ebook_marketing/add_action_report_time_enum_to_adinsights_class
…o_adinsights_class' of github.com:pnavarromuttdata/airbyte-pan-mutt-data into source_facebook_marketing/add_action_report_time_enum_to_adinsights_class
Before Merging a Connector Pull RequestWow! What a great pull request you have here! 🎉 To merge this PR, ensure the following has been done/considered for each connector added or updated:
If the checklist is complete, but the CI check is failing,
|
…ebook_marketing/add_action_report_time_enum_to_adinsights_class_seccond_attempt
…ebook_marketing/add_action_report_time_enum_to_adinsights_class_seccond_attempt
…ebook_marketing/add_action_report_time_enum_to_adinsights_class_seccond_attempt
/test connector=connectors/source-facebook-marketing
Build FailedTest summary info:
|
…t_time_enum_to_adinsights_class_seccond_attempt
@lazebnyi thanks to you for the review! |
…ebook_marketing/add_action_report_time_enum_to_adinsights_class_seccond_attempt
…ebook_marketing/add_action_report_time_enum_to_adinsights_class_seccond_attempt
…t_time_enum_to_adinsights_class_seccond_attempt
…ebook_marketing/add_action_report_time_enum_to_adinsights_class_seccond_attempt
…o_adinsights_class_seccond_attempt' of github.com:pnavarromuttdata/airbyte-pan-mutt-data into source_facebook_marketing/add_action_report_time_enum_to_adinsights_class_seccond_attempt
…ebook_marketing/add_action_report_time_enum_to_adinsights_class_seccond_attempt
…ebook_marketing/add_action_report_time_enum_to_adinsights_class_seccond_attempt
@lazebnyi are you waiting on the contributor here? |
…ebook_marketing/add_action_report_time_enum_to_adinsights_class_seccond_attempt
…t_time_enum_to_adinsights_class_seccond_attempt
@bmcgilli No, I have some issues with testing changes with CI. After successful tests, the PR will be merged. |
…t_time_enum_to_adinsights_class_seccond_attempt
…t_time_enum_to_adinsights_class_seccond_attempt
…ts class (airbytehq#26345) * chore: bump version * feat: new action_report_time spec into custom_insights stream * feat: airbyte-config new action_report_time enum spec into custom_insights stream FBMarketing * feat: new action_report_time tests * fix: update allOf instead of array json spec * chore: update docs source connector * update pr id into version history doc * update metadata.yaml * adding action report time into cloud registry and oss registry * fix formatting of python files * fix: update type action_report_time * fix: update title action report time * fix: format python files with flake8 * fix: update version to minor version * fix: remove action report time default value in test_get_custom_insights_stream * fix: keep old config for some unit tests * fix: put missing action breakdowns * fix: remove default value variable and set into class argument directly * chore: bump version * Delete changes from cloud_registry * Removed action_report_time from oss registry --------- Co-authored-by: Juan Marchese <juan.marchese@muttdata.ai> Co-authored-by: sajarin <sajarindider@gmail.com> Co-authored-by: Serhii Lazebnyi <53845333+lazebnyi@users.noreply.github.com> Co-authored-by: Serhii Lazebnyi <serhii.lazebnyi@globallogic.com>
What
In the
AdInsights
class is not available theaction_report_time
enumeration as the Facebook Marketing docs suggest.Issue
How
Add new
action_report_time
attribute toAdInsights
class, so when someone creates a new source connector, it could specify the customaction_report_time
needed (mixed
,conversion
orimpression
) having as a default valuemixed
.Recommended reading order
🚨 User Impact 🚨
By default, it will have the same behaviour as is (
action_report_time
current default value ismixed
according to this FB docs).Pre-merge Checklist
Expand the relevant checklist and delete the others.
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 hereTests
Unit
Put your unit tests output here.
-- Docs: https://docs.pytest.org/en/stable/warnings.html
Results (1.35s):
119 passed
Integration
Put your integration tests output here.
Acceptance
Put your acceptance tests output here.