dehnavi / get-simple-cms

Automatically exported from code.google.com/p/get-simple-cms
GNU General Public License v3.0
0 stars 0 forks source link

Upgrade check confusion #335

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
I think we should make some minor changes to the update check logic.

Right now it says "Upgrade Needed", which is slightly confusing.

We should add a new update check level for

"Upgrade Needed", for actually needed security and vuln releases.  Although 
"Upgrade Recommended" is probably better terminology.
and
"Upgrade Available" for normal releases.

Also we should remember to keep lang strings non context sensitive so we can 
use them in more contexts.
"UPG_NEEDED"                =>  "Upgrade needed to",
lose the "to" for example.

So maybe

vcheck ? = "UPG_RECOMMENDED"    =>  "Upgrade Recommended",
Red

vcheck 0 = "UPG_AVAILABLE"  =>  "Upgrade Available",
Yellow

With ? being a new api result for security flagged upgrades.

Original issue reported on code.google.com by tablatronics on 9 Jul 2012 at 3:12

GoogleCodeExporter commented 9 years ago
r744 changed UPG_NEEDED to 'Upgrade Recommended', did not change token id.

Original comment by tablatronics on 18 Jul 2012 at 3:54

GoogleCodeExporter commented 9 years ago

Original comment by tablatronics on 18 Jul 2012 at 4:13