Chemscribbler / aesopstables

3 stars 10 forks source link

Major Code Reorganization Effort #15

Closed LukeCroteau closed 1 year ago

LukeCroteau commented 1 year ago

One of the downsides of using Flask "out of the box default" is that it's very easy to make a couple of messes without realizing it.

This is a quick first pass at modifying the Tournaments information, and separating the Business Logic from the Data Models.

There are some Downsides to this

The upsides here are pretty big, however

LukeCroteau commented 1 year ago

If there's a file I haven't changed, at this point, I want it shot!

LukeCroteau commented 1 year ago

A note: I didn't fully clean up all the Routes yet. This PR covers about 95% of the work that needs to be done. I stopped doing routes because... I think there's a second wave of reorganization that probably needs to happen, to shift some of the render_template endpoints into a views segment.

At some point in the future, I'd like to remove some of the really hard render_template items to be Restful. Instead of embedding Python code into the html templates, I'd rather that happens as asynchronus Javascript. It works better for clients, and makes pages feel responsive.

LukeCroteau commented 1 year ago

Commit 07b5fb8 fixes #16