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 complete_oauth_server_output_specification
to spec
#27201
🐛 Source Facebook Marketing: add complete_oauth_server_output_specification
to spec
#27201
Conversation
/test connector=connectors/source-facebook-marketing
Build PassedTest summary info:
|
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,
|
complete_oauth_server_output_specification
to speccomplete_oauth_server_output_specification
to spec
/publish connector=connectors/source-facebook-marketing pre-release=true run-tests=false
|
/legacy-publish connector=connectors/source-facebook-marketing pre-release=true run-tests=false
if you have connectors that successfully published but failed definition generation, follow step 4 here |
@arsenlosenko It looks like Let me know if you need a hand with that :) |
@bnchrch good point, I'll run this file through formatter once again before merging this, thanks👍 |
…g-fix-advanced-auth
source-facebook-marketing test report (commit
|
Step | Result |
---|---|
Validate airbyte-integrations/connectors/source-facebook-marketing/metadata.yaml | ✅ |
Connector version semver check. | ✅ |
Connector version increment check. | ✅ |
QA checks | ✅ |
Connector package install | ✅ |
Build source-facebook-marketing docker image for platform linux/x86_64 | ✅ |
Unit tests | ✅ |
Integration tests | ✅ |
Acceptance tests | ❌ |
Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command
airbyte-ci connectors --name=source-facebook-marketing test
@arsenlosenko please do not use /legacy-publish to publish pre-release connector as the build logic is different now. What you publish with /legacy-test might not match what eventually gets published. |
@alafanechere understood, thanks for pointing that out |
source-facebook-marketing test report (commit
|
Step | Result |
---|---|
Validate airbyte-integrations/connectors/source-facebook-marketing/metadata.yaml | ✅ |
Connector version semver check. | ✅ |
Connector version increment check. | ✅ |
QA checks | ✅ |
Connector package install | ✅ |
Build source-facebook-marketing docker image for platform linux/x86_64 | ✅ |
Unit tests | ✅ |
Integration tests | ✅ |
Acceptance tests | ✅ |
Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command
airbyte-ci connectors --name=source-facebook-marketing test
What
Resolving this OC issue:
https://github.com/airbytehq/oncall/issues/2239
After changes made previously here, some users were not able to use connector properly due to missing entry in
advanced_auth
declaration.How
Update
advanced_auth
declaration in spec and addcomplete_oauth_server_output_specification
entry.