zpanel / zpanelx

ZPanel is a web hosting control panel written in PHP for Windows and *NIX host OS's.
http://www.zpanelcp.com
Other
415 stars 297 forks source link

Alerts fix + RoundCube update #186

Closed jacobgelling closed 10 years ago

jacobgelling commented 10 years ago

This pull request fixes #177 by adding some missing bootstrap javascript to the login page.

jacobgelling commented 10 years ago

Just updated RoundCube aswell :wink: The current config files work fine with it. No SQL patches required either.

lottfy commented 10 years ago

Not testing round-cube yet so good to know this point incase forget it

WARNING: when upgrading from <= 0.9 the old configuration files named main.inc.php and db.inc.php are now deprecated and should be replaced with one single config.inc.php file. Run the ./bin/update.sh script to get this conversion done or manually merge the files. NOTE: the new config.inc.php should only contain options that differ from the ones listed in defaults.inc.php.

jacobgelling commented 10 years ago

Regarding the RoundCube config files - they do still work but it is now just best practice to use one config file. I'm not sure of all the changes that would be needed (such as installers etc) and what ZPanel has changed from the default RoundCube config files.

satdav commented 10 years ago

Can I ask you update the roundcube to the latest build

jacobgelling commented 10 years ago

@satdav I've already updated to the latest stable version :wink: Just waiting for the update to be approved.

lottfy commented 10 years ago

@jacobg830 you must work to apply this PR to installer as .sh

jacobgelling commented 10 years ago

@lottfy What to you mean applying this installer as .sh? And what guide?

I don't think any install/upgrade changes are required as the SQL files are the same.

Edit: I see you made some commits on your fork - do they fix the problems?

lottfy commented 10 years ago

meaning who will update to e.g 10.1.2 need simple script to apply .sql changes and also config WARNING: when upgrading from <= 0.9 the old configuration files named main.inc.php and db.inc.php are now deprecated and should be replaced with one single config.inc.php file. Run the ./bin/update.sh script to get this conversion done or manually merge the files. NOTE: the new config.inc.php should only contain options that differ from the ones listed in defaults.inc.php. go to https://github.com/zpanel/installers and open new PR WITH .SH AND CONFIG FILES

jacobgelling commented 10 years ago

@lottfy Right but there were no SQL changes and the old config files still work but are deprecated.

satdav commented 10 years ago

@bobsta63 can we please get your throughts on this update as I think what it is ready from my side to go live but th @lottfy is disagreeing with it

jacobgelling commented 10 years ago

Ok I get what you mean now. I'll hopefully post a new PR with a merged config file and the sh file soon. Sorry for the confusion haha.

Note to others: this pull request still works perfectly fine but doesn't use the new style of Roundcube config files (yet).