AskUploadButton renders as "0" due to isAsk sometimes being zero

[petri]

Describe the bug

With chat history using SQLAlchemy data layer with SQLite, the UI becomes like this (see the zeros):

This is because Message.tsx renders following into zero when isAsk is zero and isRunning is falsy:

{!isRunning && isAsk && (
<AskUploadButton onError={onError} />
)}

That zero value originates from SQLAlchemy layer implementation. Perhaps there should be casting to boolean there.

In my humble opinion, the above UI logic is not optimal either, as evidenced by this issue. There's nothing wrong with preferring readability and simplicity over cleverness and reliance on what could be here considered a side effect.

To Reproduce

Implement chat history with SQLAlchemy & SQLite.

[dokterbob]

I agree on explicit vs. implicit (and you're probably right about the SQLAlchemy part - there's more than a few things to be improved there).

Would you please be so kind as to shoot a PR with 0786e17?

[petri]

I agree on explicit vs. implicit (and you're probably right about the SQLAlchemy part - there's more than a few things to be improved there).

Would you please be so kind as to shoot a PR with 0786e17?

@dokterbob there you go.

[willydouhard]

Should be fixed in the latest release!