Fixed issue with bubbles + fixed french translation issue

app/src/main/java/org/linphone/compatibility/Api29Compatibility.kt

@@ -20,6 +20,7 @@
 package org.linphone.compatibility
 
 import android.annotation.TargetApi
+import android.app.NotificationChannel
 import android.app.NotificationManager
 import android.content.ContentValues
 import android.content.Context
@@ -30,6 +31,7 @@ import android.provider.MediaStore
 import android.view.View
 import android.view.contentcapture.ContentCaptureContext
 import android.view.contentcapture.ContentCaptureSession
+import androidx.core.app.NotificationManagerCompat
 import org.linphone.R
 import org.linphone.core.ChatRoom
 import org.linphone.core.Content
@@ -41,6 +43,24 @@ import org.linphone.utils.LinphoneUtils
 @TargetApi(29)
 class Api29Compatibility {
     companion object {
+        fun createMessageChannel(
+            context: Context,
+            notificationManager: NotificationManagerCompat
+        ) {
+            // Create messages notification channel
+            val id = context.getString(R.string.notification_channel_chat_id)
+            val name = context.getString(R.string.notification_channel_chat_name)
+            val description = context.getString(R.string.notification_channel_chat_name)
+            val channel = NotificationChannel(id, name, NotificationManager.IMPORTANCE_HIGH)
+            channel.description = description
+            channel.lightColor = context.getColor(R.color.notification_led_color)
+            channel.enableLights(true)
+            channel.enableVibration(true)
+            channel.setShowBadge(true)
+            channel.setAllowBubbles(true)
+            notificationManager.createNotificationChannel(channel)
+        }
+
         fun extractLocusIdFromIntent(intent: Intent): String? {
             return intent.getStringExtra(Intent.EXTRA_LOCUS_ID)
         }

app/src/main/java/org/linphone/compatibility/Compatibility.kt

@@ -84,7 +84,11 @@ class Compatibility {
                 Api26Compatibility.createServiceChannel(context, notificationManager)
                 Api26Compatibility.createMissedCallChannel(context, notificationManager)
                 Api26Compatibility.createIncomingCallChannel(context, notificationManager)
-                Api26Compatibility.createMessageChannel(context, notificationManager)
+                if (Version.sdkAboveOrEqual(Version.API29_ANDROID_10)) {
+                    Api29Compatibility.createMessageChannel(context, notificationManager)
+                } else {
+                    Api26Compatibility.createMessageChannel(context, notificationManager)
+                }
             }
         }
 

app/src/main/java/org/linphone/notifications/NotificationsManager.kt

@@ -737,9 +737,11 @@ class NotificationsManager(private val context: Context) {
                 .setDeleteIntent(getMarkMessageAsReadPendingIntent(notifiable))
         }
 
-        if (Compatibility.canChatMessageChannelBubble(context)) {
+        if (!Compatibility.canChatMessageChannelBubble(context)) {
-            notificationBuilder.bubbleMetadata = bubble
+            Log.w("[Notifications Manager] This conversation wasn't granted bubble permission yet")
         }
+        // We still need to set the bubbleMetadata, otherwise user won't ever be able to enable bubbles!
+        notificationBuilder.bubbleMetadata = bubble
         return notificationBuilder.build()
     }
 

app/src/main/res/values-fr/strings.xml

@@ -553,8 +553,8 @@
     <string name="content_description_toggle_audio_menu">Afficher ou masquer le menu de sélection du périphérique audio</string>
     <string name="chat_message_notification_hidden_content">&lt;Censuré&gt;</string>
     <string name="content_description_enable_speaker">Envoyer le son vers le haut-parleur</string>
-    <string name="call_settings_overlay_title">Superposition des notifications push sur l\'écran</string>
+    <string name="call_settings_overlay_title">Notification d\'appel flottante</string>
-    <string name="call_settings_overlay_summary">L\'autorisation d\'afficher les notifications push en superposition est requise</string>
+    <string name="call_settings_overlay_summary">Une autorisation explicite est requise</string>
     <string name="chat_settings_mark_as_read_notif_dismissal_title">Marquer comme lu lorsque la notification est supprimée</string>
     <string name="chat_settings_launcher_shortcuts_summary">Seront remplacés par les raccourcis de contacts si activés</string>
     <string name="account_settings_transport_title">Transport</string>