Android: <br/> tags are not removed from announcement texts

[j-ed]

Actual behaviour

If you've created an announcement using the announcement center app, carriage returns (CR) are not correctly displayed in the Nextcloud app. Instead <br/> is shown in the announcement text.

Expected behaviour

Carriage returns tags <br/> should be handled and displayed correctly in an announcement text within the Nextcloud app.

Steps to reproduce

1. Create an announcement text using the announcement center app, which includes carriage returns.
2. Open the Nextcloud app on your mobile device and check the displayed activities
   -> You should see <br/> tag in the previously created announcement text.

Environment data

Android version: 8.0
Device model: Samsung Galaxy S7
Stock or customized system: Stock ROM
Nextcloud app version: 3.4.2
Nextcloud server version: 15.0.2

Logs

Web server error log

No error messages were logged.

Nextcloud log (data/nextcloud.log)

No error messages were logged.

[nextcloud-android-bot]

GitMate.io thinks possibly related issues are https://github.com/nextcloud/android/issues/2208 (Log_OC.d(TAG, e.getMessage()) -> getMessage() can be null), https://github.com/nextcloud/android/pull/2944 (remove dividers), https://github.com/nextcloud/android/issues/59 (Remove mail_feedback), https://github.com/nextcloud/android/issues/940 (Remove SuppressLint), and https://github.com/nextcloud/android/issues/655 (Rename "remove" to "delete").

[AndyScherzinger]

@nickvergessen @tobiasKaminsky is this something the app should handle or wouldn't this rather be considered a server-side issue since the server should sent a linebreak, not the html equivalent?

[nickvergessen]

I will check this tomorrow, maybe I broke something while adding markdown support to the announcements app.

[nickvergessen]

We will fix this in the other apps, I think the android app is doing everything correct here:
https://github.com/nextcloud/server/pull/13819