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

Run sharing in main thread #1090

Merged
merged 1 commit into from
Jul 14, 2020
Merged

Run sharing in main thread #1090

merged 1 commit into from
Jul 14, 2020

Conversation

TdoubleG
Copy link

@TdoubleG TdoubleG commented Apr 3, 2020

In Xcode you will see an error that the ActivityController should run in main thread

@murcie
Copy link

murcie commented Apr 21, 2020

Same issue. I think should be merged.

[reports] Main Thread Checker: UI API called on a background thread: -[UIViewController setValue:forKey:]
PID: 56663, TID: 4051744, Thread name: (none), Queue name: com.apple.root.default-qos, QoS: 0
Backtrace:
4   ProjectName                         0x000000010494f8c8 __61-[SocialSharing shareInternal:withOptions:isBooleanResponse:]_block_invoke + 2276

@koraysels
Copy link

could this be the cause of the long initial load time of the share menu?

@TdoubleG
Copy link
Author

Yes

@koraysels
Copy link

cool man, ill just use your branch in my app :D

@santiq
Copy link
Contributor

santiq commented Jul 13, 2020

Instead of deleting the line, this should be a configuration option

@EddyVerbruggen EddyVerbruggen merged commit 5da61cf into EddyVerbruggen:master Jul 14, 2020
@EddyVerbruggen EddyVerbruggen added this to the 5.6.9 milestone Jul 14, 2020
@EddyVerbruggen
Copy link
Owner

Everybody, please promise me nobody is going to open an issue that Cordova is yelling in the console that this method is taking too long and should not run in the main thread.

@TdoubleG
Copy link
Author

@santiq it doesn’t make sense to set the line as configuration because it’s an error from the iOS System. You can find it also in the documentation from Apple

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

Successfully merging this pull request may close these issues.

5 participants