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

Bug(?) with passing to upload.farm for rendering and rate limiter #26

Closed
Laukei opened this issue Feb 12, 2017 · 1 comment
Closed

Bug(?) with passing to upload.farm for rendering and rate limiter #26

Laukei opened this issue Feb 12, 2017 · 1 comment

Comments

@Laukei
Copy link

Laukei commented Feb 12, 2017

Hi Henrik, I hope things are well!

I got an email from someone today who has a problem with how upload.farm handles his save. The bug involved is relatively simple to fix, but he also mentioned that

On the planner, when trying to render w/ a different season, I receive
this error:

stardew.info say: Rendering failed. Please try again in few minutes

I loaded his savefile into the planner and rendered it in a couple of different seasons, which worked fine, so I tried to figure out what the bug could be he was getting. The only thing I can think is that he might be hitting the upload.farm plan rendering API rate limiter. If a render hits the rate limiter, does it have a separate alert to the user, or does it just time out and say rendering failed? If it does this, would it be possible to give a different error when it times out, maybe mentioning that upload.farm is rate limiting and to try again later? (I hope that's not too demanding, apologies if it is...)

Cheers!
Rob

@hpeinar
Copy link
Owner

hpeinar commented Feb 13, 2017

I've added separate error message when upload.farm API responds with status code 429 (rate limited).

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

No branches or pull requests

2 participants