Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open photo editor from video editor #76

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

GlebPBanuba
Copy link
Contributor

No description provided.

@dhyaan-bigshorts
Copy link

I encountered an issue with the exported path not being sent to Flutter, and the flutterResult not being correctly passed back to Flutter. To address this, I made the following modifications:

Added flutterResult Parameter:
Included the flutterResult parameter in the presentPhotoEditor and related function calls to ensure the result is correctly passed back to Flutter.

Ensured Proper Result Handling:
Updated methods like photoEditorDidFinishWithImage and videoEditorDidCancel to properly handle and send the result using flutterResult.

func presentPhotoEditor(with launchConfig: PhotoEditorLaunchConfig,
                        flutterResult: @escaping FlutterResult) {
    self.flutterResult = flutterResult
    photoEditorSDK?.presentPhotoEditor(
        withLaunchConfiguration: launchConfig,
        completion: nil
    )
}

Pass the same parameters from other functions, i.e., checkLicenseAndOpenPhotoEditor.

This ensures that the flutterResult is properly passed and handled in all related function calls, facilitating correct communication between the native code and Flutter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants