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

[Improvement suggestion] - Queue articles publication to meet forem rate limiting #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PChol22
Copy link

@PChol22 PChol22 commented Jul 10, 2023

Hi!

I am a big fan and user of dev-to-git!

My articles repository contains like 20 articles, and it's starting to hit API rate limits when dev-to-git is publishing everything. I have to restart the job multiple times if I want my new articles to be eventually published.

image

I went through forem's documentation, and it seems like the quotas are:

  • 30 calls/30sec for article update
  • 10 calls/30sec for article creation

I think that we could use a small queuing lib designed to stay under the 10calls/30sec rate limit when publishing articles.
-> It would not impact people publishing a small amount of articles that much, and would really help big repositories like mine!

I wrote a bit of code that should fix the issue, WDYT? I am open to suggestions and modifications of course.

@maxime1992
Copy link
Owner

Hi there, thanks for the kind words an the PR!

That said, I'm in the middle of a complete rewrite 😅 ! Nothing I wanted to mention before it was done to avoid pressure but I feel like I've got to disclose it now haha.

So I'll put this on hold for now and try to take it into account in the new version directly.

@PChol22
Copy link
Author

PChol22 commented Jul 12, 2023

Ok it's totally fine for me! Good luck for your rewrite!

@PChol22 PChol22 closed this Jul 12, 2023
@maxime1992
Copy link
Owner

maxime1992 commented Jul 15, 2023

I went through forem's documentation, and it seems like the quotas are:
30 calls/30sec for article update
10 calls/30sec for article creation

Where did you find those info? I looked here: https://developers.forem.com/api/v1

But couldn't find it 🤔

@maxime1992 maxime1992 reopened this Jul 15, 2023
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