Merge pull request #12874 from woocommerce/12793-receipt-is-not-attac…

…hed-to-text-message

[Fix] Receipt is not attached to text message

RELEASE-NOTES.txt

@@ -3,7 +3,7 @@
 *** For entries which are touching the Android Wear app's, start entry with `[WEAR]` too.
 21.1
 -----
-
+- [*] Receipts can be shared via a google's sms application now [https://github.com/woocommerce/woocommerce-android/pull/12874]
 
 21.0
 -----

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/payments/receipt/PaymentReceiptShare.kt

@@ -44,7 +44,7 @@ class PaymentReceiptShare @Inject constructor(
             selectedSite.get().name.orEmpty()
         )
         val intent = Intent(Intent.ACTION_SEND).apply {
-            type = "application/*"
+            type = "text/plain"
             putExtra(Intent.EXTRA_STREAM, uri)
             putExtra(Intent.EXTRA_TEXT, text)
         }
@@ -61,11 +61,11 @@ class PaymentReceiptShare @Inject constructor(
     }
 
     sealed class ReceiptShareResult {
-        object Success : ReceiptShareResult()
+        data object Success : ReceiptShareResult()
         sealed class Error : ReceiptShareResult() {
             data class Sharing(val exception: Exception) : Error()
-            object FileCreation : Error()
-            object FileDownload : Error()
+            data object FileCreation : Error()
+            data object FileDownload : Error()
         }
     }
 }

WooCommerce/src/main/kotlin/com/woocommerce/android/util/FileDownloader.kt

@@ -15,7 +15,6 @@ class FileDownloader @Inject constructor(
      *
      * @return true if the download succeeded and false otherwise
      */
-    @Suppress("BlockingMethodInNonBlockingContext")
     suspend fun downloadFile(url: String, destinationFile: File): Boolean {
         return try {
             if (destinationFile.exists()) destinationFile.delete()