← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands-website/cleanup_ggz into lp:widelands-website

 

Review: Resubmit

I found a failure due to a missing import in this branch and fixed it. The culprit why i not stumbled over this the first time was a remaining fields.pyc file which i haven't deleted when removing fields.py. So the missing import doesn't show up in my local branch... lesson learned... I guess this wouldn't show up on the server as well, because the *.pyc files wouldn't be removed when  bzr pull is executed... This could lead into unexpected behavior i think. Maybe we should delete every *.pyc file after stopping the website? Something like 

find . -name '*.pyc' -delete

Any suggestions to this?
---------------------------------
The updated branch fixes the missing import and also adds a migration to remove the fields 'last_login' and 'confirmed' (including needed adjustments in wlggz.admin. When running 

./manage.py migrate

the fields in the table wlggz_ggzauth in the database get removed (deleted).

The comment for EMAIL_BACKEND is completed with:

# Don't use this on the widelands server!

If someone says: 'Go' i merge this, make a test on the alpha site and after all is fine i will update wl.widelands.org :-)
-- 
https://code.launchpad.net/~widelands-dev/widelands-website/cleanup_ggz/+merge/301588
Your team Widelands Developers is subscribed to branch lp:widelands-website.


References