rpherbig / dr-scripts

A series of Lich 5 (https://github.com/elanthia-online/lich-5) scripts for use with DragonRealms (http://www.play.net/dr/). Donations are welcome (http://www.paypal.me/rcuhljr)!
GNU General Public License v2.0
54 stars 178 forks source link

[scripts][dependency] Add messaging for forced script downloads #6973

Closed MahtraDR closed 1 week ago

MahtraDR commented 2 weeks ago

Adds messaging when scripts are force-checked, and adds the option to force-check all scripts.

Resolves https://github.com/rpherbig/dr-scripts/issues/6970

Kiriawen commented 2 weeks ago

usage would be ;eq force_refresh_scripts ?

MahtraDR commented 2 weeks ago

usage would be ;eq force_refresh_scripts ?

There are two commands.

One - Force-gets all scripts (I am not sure we actually want this, we may only want it to CHECK all scripts, and get the ones that are out of date?)

;eq force_refresh_scripts

Two - force-gets one script

;eq get_script('script-name')

Note this should resolve https://github.com/rpherbig/dr-scripts/issues/6970 however we end up tuning it.

Kiriawen commented 2 weeks ago

yeah having it check for updates would be better than redownloading everything. Save on bandwidth at least

Kiriawen commented 2 weeks ago

though having both a way to force a redownload of everything and a way to just check for updated files wouldn't be a bad idea either.

MahtraDR commented 2 weeks ago

though having both a way to force a redownload of everything and a way to just check for updated files wouldn't be a bad idea either.

The way it works now, it'll check every file and redownload if they're out of date or don't exist. Is that good enough here?

Kiriawen commented 1 week ago

yep, that works. That's what I was expecting.