Code review comment for lp:~trb143/openlp/bug-850254

Revision history for this message
Jonathan Corwin (j-corwin) wrote :

Too many blank lines 56-58
There is an elif on 53 but an if on 55.

I'm not sure about the term "status" in httpserver, it doesn't seem descriptive enough to me, as lots of things could have a status. Perhaps displaystatus or something?

review: Needs Fixing

« Back to merge proposal