@ruKurz I could have also validated that there is a String provided for the same result.
However, as an empty String ("") is evaluated as false in JS, I opted for the inverse.-
wdyt?

As non JS expert I would say:

if (isNotEmptyOrWhitespaceOnly(inputValue)) {
  // save & close
} else {
  // cancel
}

I think there should be three cases handled when closing a conversation:

1. Close and store
2. Close and do not store
3. Do not close / cancel the dialog

As far as I understand the code, now it is possible to

1. close and store
2. cancel the dialog

@mrsimpson WDYT?

@ruKurz what's the business value of "Close and do not store"
Would this be an explicit decision by the user?

Yes, I think the "explicit decision" is a benefit. But if you aim to grab as much data with Smarti as possible the "close and do not store option" would cause the lose of conversational data.

So the trade-off is "user control" vs. "data octopus".

@ruKurz I extracted the check into a separate method. Any objections merging it?

@ruKurz whether we provide a "close but don't store" is another decision, imho.