Fix #181

[thw26]

Fix #181

The second commit should help prepare for #168.

[thw26]

We need to clear the recently added TODO.

[n8marti]

Here's the conflict I see if I start a merge locally:

<<<<<<< HEAD
def cli_acknowledge_question(question_text, no_text, secondary):
    if not cli_question(question_text, secondary):
        logos_msg(no_text)
=======
def cli_acknowledge_question(QUESTION_TEXT, NO_TEXT):
    if not cli_question(QUESTION_TEXT, ""):
        logos_msg(NO_TEXT)
>>>>>>> 181-regression
        return False
    else:
        return True

What's the correct fix? It looks like it's only used once:

$ grep -n cli_acknowledge_question *.py
msg.py:229:def cli_acknowledge_question(question_text, no_text, secondary):
msg.py:271:        return cli_acknowledge_question(question_text, no_text, secondary)

Can we just keep what's already main and drop what was changed in 181-regression?