feat(feedback): Use only image uri in the onAddScreenshot callback (#4546)

* Save form state for unsubmitted data

* Show selected screenshot

* Use image uri instead of UInt8Array in  onAddScreenshot callback

* Omit isVisible from state

* Save/clear form state on unmount

* Pass the missing attachment parameter in the onSubmitSuccess

* Use only the uri parameter for the onAddScreenshot callback

* Use instance variable for _didSubmitForm

* Fixed callback function parameter name for clarity

Co-authored-by: Krystof Woldrich <[email protected]>

* Fixes lint issue

---------

Co-authored-by: Krystof Woldrich <[email protected]>

Author: antonis

packages/core/src/js/feedback/FeedbackWidget.tsx

@@ -158,9 +158,17 @@ export class FeedbackWidget extends React.Component<FeedbackWidgetProps, Feedbac
       } else {
         // Defaulting to the onAddScreenshot callback
         const { onAddScreenshot } = { ...defaultConfiguration, ...this.props };
-        onAddScreenshot((filename: string, attachement: Uint8Array) => {
-          // TODO: Add support for image uri when using onAddScreenshot
-          this.setState({ filename, attachment: attachement, attachmentUri: undefined });
+        onAddScreenshot((uri: string) => {
+          NATIVE.getDataFromUri(uri).then((data) => {
+            if (data != null) {
+              this.setState({ filename: 'feedback_screenshot', attachment: data, attachmentUri: uri });
+            } else {
+              logger.error('Failed to read image data from uri:', uri);
+            }
+          })
+          .catch((error) => {
+            logger.error('Failed to read image data from uri:', uri, 'error: ', error);
+          });
         });
       }
     } else {

packages/core/src/js/feedback/FeedbackWidget.types.ts

@@ -171,7 +171,7 @@ export interface FeedbackCallbacks {
   /**
    * Callback when a screenshot is added
    */
-  onAddScreenshot?: (attachFile: (filename: string, data: Uint8Array) => void) => void;
+  onAddScreenshot?: (addScreenshot: (uri: string) => void) => void;
 
   /**
    * Callback when feedback is successfully submitted

packages/core/src/js/feedback/defaults.ts

@@ -33,7 +33,7 @@ export const defaultConfiguration: Partial<FeedbackWidgetProps> = {
       );
     }
   },
-  onAddScreenshot: (_: (filename: string, data: Uint8Array) => void) => {
+  onAddScreenshot: (_: (uri: string) => void) => {
     if (__DEV__) {
       Alert.alert('Development note', 'onAddScreenshot callback is not implemented.');
     }