Various comments fixes

[skjnldsv]

• Prevent submitting empty messages
• Fix white-spacing
• Hide long messages like it was before (click to unroll)
• Fix undo timeout by bumping the nextcloud dialogs
• Add mentions data
  

Follow-up #23173

[skjnldsv]

/compile /

[blizzz]

from the screenshots, does it fix a bug where the display name was not shown, or does it now show the internal user id?

[skjnldsv]

from the screenshots, does it fix a bug where the display name was not shown, or does it now show the internal user id?

I have no idea what bug you are refering to. We're now using the data provided by the comments>mentions dav properties. SO it should render as it used to :)

[blizzz]

from the screenshots, does it fix a bug where the display name was not shown, or does it now show the internal user id?

I have no idea what bug you are refering to. We're now using the data provided by the comments>mentions dav properties. SO it should render as it used to :)

I think I misinterpreted the screenshot from the description. Never mind :)

[skjnldsv]

/compile amend /

[skjnldsv]

/compile amend /

[skjnldsv]

Jsunit tests fixed

[faily-bot]

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 34676: failure

mysql8.0-php7.4

Show full log

There were 2 warnings:

1) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

2) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

--

There was 1 failure:

1) Test\Share20\DefaultShareProviderTest::testGroupDeleted with data set #3 (array(1, 'group1', array('foo', 'bar')), 'group2', false)
Failed asserting that actual size 2 matches expected size 3.

/drone/src/tests/lib/Share20/DefaultShareProviderTest.php:2418