-
Notifications
You must be signed in to change notification settings - Fork 87
Conversation
@casperstorm Test this out. I've currently only implemented "checking" for new updates. It's hard-coded to check every 30 minutes. Should we make that configurable or is 30mins sane? Considering how slow to update Curse API is, not sure that matters too much. Adding the ability to apply these updates automatically will be trivial. I'll make sure that's opt-in via config. When an update is found for the Flavor, it'll resort the addons by status to push those updates to the top of the screen. |
This is a great PR, @tarkah. It looks great.
I have thought about it for a while and initially I wanted to make it quite customisable with a drop-down of different intervals, but then again... why? As you also mention we are limited by the API and 30mins should be MORE than enough and would simplify the GUI (and UX) a lot. I suggest we add a checkbox to enable "auto update" and enable auto refresh for everyone. The auto refresh i feel is a QOL thing everyone would benefit from. |
@casperstorm I 100% agree with you. Worst case, we get people asking for control and we can add the interval down the road. |
chore(bot): updated localizations
@tarkah I've removed the test in Other than that all is LGTM! It has been tested and works. |
Proposed Changes
Checklist
CHANGELOG.md
if knowledge of this change could be valuable to users