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

Bump facebook API to v14.0 #201

Merged
merged 3 commits into from
Oct 20, 2022
Merged

Bump facebook API to v14.0 #201

merged 3 commits into from
Oct 20, 2022

Conversation

shantanu73
Copy link
Contributor

@shantanu73 shantanu73 commented Oct 12, 2022

Bumps the facebook api version to v14.0

Description of change

(write a short description or paste a link to JIRA)

Manual QA steps

Risks

Rollback steps

  • revert this branch

CHANGELOG.md Outdated
@@ -1,5 +1,8 @@
# Changelog

## 1.18.4
* Facebook busines API to v15.0 [#201](https://github.com/singer-io/tap-facebook/pull/201)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You bumped it to v14.0, not v15.0

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cosimon I went through the link : https://pypi.org/project/facebook-business/
Apparently latest sdk version is 14.0.0 and latest api version is 15.0, their sdk is not up to date with latest api version.
Also, I confirmed that versions previous to v15.0 are deprecated. How shall we proceed with this ?

@shantanu73 shantanu73 changed the title Bump facebook API to v15.0 Bump facebook API to v14.0 Oct 17, 2022
@shantanu73 shantanu73 merged commit ecf46e5 into master Oct 20, 2022
jamjensen added a commit to jamjensen/tap-facebook that referenced this pull request Nov 20, 2022
v15 was released on 10/25, five days after singer-io#201 was approved to bump to v14
rarce added a commit to Datalized/tap-facebook that referenced this pull request Feb 6, 2023
jesuejunior added a commit to sixcodes/tap-facebook that referenced this pull request Mar 17, 2023
* Bump facebook API to v15.0 which is used in sdk facebook_business version 14.0.0

* New tap-facebook bump version 1.18.4

* changed Facebook API version to v14.0

* Bump Facebook API to v15.0

v15 was released on 10/25, five days after singer-io#201 was approved to bump to v14

---------

Co-authored-by: shantanu73 <shantanudhiman25@gmail.com>
Co-authored-by: jamjensen <jamesronaldjensen@gmail.com>
jesuejunior pushed a commit to sixcodes/tap-facebook that referenced this pull request Mar 17, 2023
* Bump facebook API to v14.0 which is used in sdk facebook_business version 14.0.0

* New tap-facebook bump version 1.18.4
jesuejunior added a commit to sixcodes/tap-facebook that referenced this pull request Mar 28, 2023
* Add country to ads insights country breakdown PK

* Remove `ads.creative.creative_id` in favor of `ads.creative.id`

* Added missing test cases

* Updated pagination test case

* Removed pagination test case

* Added back pagination test file

* conversions

* feature - add conversion to all streams

* Jamjensen patch 1 (#8)

* Bump facebook API to v15.0 which is used in sdk facebook_business version 14.0.0

* New tap-facebook bump version 1.18.4

* changed Facebook API version to v14.0

* Bump Facebook API to v15.0

v15 was released on 10/25, five days after singer-io#201 was approved to bump to v14

---------

Co-authored-by: shantanu73 <shantanudhiman25@gmail.com>
Co-authored-by: jamjensen <jamesronaldjensen@gmail.com>

---------

Co-authored-by: Dan Mosora <dan@stitchdata.com>
Co-authored-by: vagrant <vagrant@vagrant.vm>
Co-authored-by: Umang Agrawal <umang.agrawal@crestdatasys.com>
Co-authored-by: Gary James <gary.james19@gmail.com>
Co-authored-by: shantanu73 <shantanudhiman25@gmail.com>
Co-authored-by: jamjensen <jamesronaldjensen@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants