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.

Sign up for GitHub

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

Jump to bottom

🎉 Source Facebook: rate limit not always handled #3525 #3600

Merged
merged 13 commits into from
May 26, 2021
Merged

🎉 Source Facebook: rate limit not always handled #3525 #3600

merged 13 commits into from
May 26, 2021

Conversation

keu
Copy link
Contributor

@keu keu commented May 25, 2021 •
edited
Loading

What

closes #3525

How

Describe the solution

Pre-merge Checklist

  • Run integration tests
  • Publish Docker images

Recommended reading order

  1. test.java
  2. component.ts
  3. the rest

@keu keu requested review from sherifnada and yevhenii-ldv and removed request for ChristopheDuong and cgardens May 25, 2021 23:24
@keu
Copy link
Contributor Author

keu commented May 25, 2021 •
edited by github-actions bot
Loading

/test connector=source-facebook-marketing

🕑 source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/876695294
❌ source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/876695294

@keu
Copy link
Contributor Author

keu commented May 25, 2021 •
edited by github-actions bot
Loading

/test connector=source-facebook-marketing

🕑 source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/876726260
❌ source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/876726260

@keu keu self-assigned this May 25, 2021
@keu keu added the zazmic label May 25, 2021
@keu
Copy link
Contributor Author

keu commented May 25, 2021 •
edited by github-actions bot
Loading

/test connector=source-facebook-marketing

🕑 source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/876758065
❌ source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/876758065

@keu
Copy link
Contributor Author

keu commented May 26, 2021 •
edited by github-actions bot
Loading

/test connector=source-facebook-marketing

🕑 source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/876823895
❌ source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/876823895

@keu
Copy link
Contributor Author

keu commented May 26, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/source-facebook-marketing

🕑 connectors/source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/876844024
❌ connectors/source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/876844024

@keu
Copy link
Contributor Author

keu commented May 26, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/source-facebook-marketing

🕑 connectors/source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/877368492
✅ connectors/source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/877368492

@keu keu merged commit 008e22e into master May 26, 2021
@keu keu deleted the keu/fb-marketing-fix-backoff branch May 26, 2021 06:09
@avaidyanatha avaidyanatha changed the title Source Facebook: rate limit not always handled #3525 🎉 Source Facebook: rate limit not always handled #3525 Jun 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sherifnada sherifnada sherifnada approved these changes

@yevhenii-ldv yevhenii-ldv Awaiting requested review from yevhenii-ldv

Assignees

@keu keu

Labels
connectors/source/facebook-marketing connectors/sources-api
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Source Facebook: rate limit not always handled
4 participants
@keu @sherifnada @karinakuz @eugene-kulak