← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~trimardio/widelands-website/scheduling into lp:widelands-website

 

Agreed on every point. I really liked your suggestion for the main page, I didn't like the result. Now it's far better!

>Is there a reason why the model field is called 'hour' but it is a DateTime field? Something like:

>if datetime.now() > a.hour:
    a.delete()


We are talking about one particular hour, not one particular date. So that's the reason. Now maybe if we used time rather than hour? Or we could use date, but isn't that confusing? It was for me at the start.

As for the avaibilities and availabilities you are correct indeed. There is no debate to have :P


-- 
https://code.launchpad.net/~trimardio/widelands-website/scheduling/+merge/330997
Your team Widelands Developers is requested to review the proposed merge of lp:~trimardio/widelands-website/scheduling into lp:widelands-website.


References