-
-
Notifications
You must be signed in to change notification settings - Fork 285
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
Pulse breaks Watch Connectivity after opening #201
Comments
Good catch. I don't know why it never occurred to me that the delegate was shared 🤦♂️
Can you please elaborate on that? There is a new |
Meaning the developer integrating Pulse is responsible for implementing a WCSession delegate and check if a file transfer is happening. |
Ah, got it. Yeah, that makes the most sense. But I'm not sure if it's worth keeping this feature given that the Btw, in addition to Pulse for Mac, there is also a new Pulse app coming for iOS. It'll allow you opening Pulse log files on the iOS devices. I haven't pushed it to TestFlight yet. |
Yeah, |
I've done a couple of (successful) prototypes, and I'm planning to:
This will not be too much work from the user's point of view and will address the issues you highlighted. Once again, thank you for testing it and reporting the detailed issues. I really appreciate it! |
Fixed in 4.0.0-beta.3. |
After opening the PulseUI on iOS a
LoggerSyncSession
is created. This class replaces the WCSession delegate on init, without any regard for an existing delegate.This will more or less break any iOS app that has a companion Watch app.
A better solution would be to either:
LoggerSyncSession
The text was updated successfully, but these errors were encountered: