Skip to content

Conversation

@duk3luk3
Copy link
Member

The server periodically fetches information like the list of featured
mods and the minimum client version from the database.

A sixty-minutes interval is not reasonable. This commit changes it to 10
minutes.

Fixes #280

@coveralls
Copy link

Coverage Status

Coverage remained the same at 78.726% when pulling 4361d25 on duk3luk3:fix/280-cron-interval into 4e04668 on FAForever:develop.

@duk3luk3
Copy link
Member Author

duk3luk3 commented May 1, 2017

@micheljung can this go into 0.8?

@duk3luk3
Copy link
Member Author

duk3luk3 commented May 2, 2017

I am using a test account to test something, which I also need to make uniqueid_exempt so I can use it.

If I do that, I have to wait on average half an hour until the server updates the uid exempt users dict and I can actually use it.
This is extremely annoying to me. It breaks my flow and it means I need to go do something else until the server updates.

There is no reason to make this interval 1 hour long.

@micheljung
Copy link
Member

Tried to merge from mobile; Server error. Please try again

The server periodically fetches information like the list of featured
mods and the minimum client version from the database.

A sixty-minutes interval is not reasonable. This commit changes it to 10
minutes.

Fixes FAForever#280
@duk3luk3 duk3luk3 force-pushed the fix/280-cron-interval branch from 4361d25 to a273a90 Compare May 3, 2017 04:57
@duk3luk3
Copy link
Member Author

duk3luk3 commented May 3, 2017

Rebased, maybe it'll work better now :-)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 78.726% when pulling a273a90 on duk3luk3:fix/280-cron-interval into 24ba17f on FAForever:develop.

@micheljung micheljung merged commit a5adc6f into FAForever:develop May 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants