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

Watch compliction #506

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Watch compliction #506

wants to merge 12 commits into from

Conversation

creepymonster
Copy link
Owner

@creepymonster creepymonster commented Jan 21, 2023

@bassrock i had quite a struggle with it until I had everything up and running again (all sorts of things with bundle identifiers).

Maybe you like to look over it once :)

Many greetings
Reimar

@bassrock
Copy link
Contributor

@creepymonster Re-looking into this, I think some things will need to be updated. Been a bit since I did watchOS apps (they now run on the watch 😅 instead of via the phone), the data can't easily be shared as I thought without implementing WatchConnectivity. https://developer.apple.com/documentation/watchconnectivity/implementing_two-way_communication_using_watch_connectivity

I'll take a look this weekend at that.

@creepymonster
Copy link
Owner Author

creepymonster commented Jan 21, 2023

Thanks a lot :)

I have no experience with the Apple Watch at all.

As for the adjustments for WatchConnectivity I was not sure. I've always read about "Two-Way Communication" in that context - we actually only have One-Way, from the phone to the Watch.

I didn't have any data from the iPhone in the simulator, though. But maybe the Shared App Group on the iPhone is only there for different apps on the Watch 🤷.

Many greetings
Reimar

@bassrock bassrock mentioned this pull request Jan 22, 2023
@bassrock
Copy link
Contributor

@creepymonster I started #508 but wcsession is pretty tricky, i'll need to do a bunch of reading up to get something reliable. May need to get back to this later.

@tzdesign
Copy link

tzdesign commented Jul 1, 2023

Hi guys,

I'm Tobi, also from Berlin.

I came across your app because the Libre lacks a bit in terms of UI.

Are you still working on the Apple Watch app? Maybe I can help you a bit. I have been through the same struggle myself.

Am I right in thinking that you use a lot of user defaults for data?

With cloudkitcontainers you could outsource the data to a framework and use it for both apps.

Otherwise there is no way around the watch connectivity. Most of the time it's important to have two-way communication just to ask the phone for data.

I will check out this PR during the week and can give you some guidance if you want.

@DAcodedBEAT DAcodedBEAT mentioned this pull request Jul 15, 2023
@robinwo
Copy link

robinwo commented Oct 28, 2023

Hope we can make this work! Any update on this Tobi?

@tzdesign
Copy link

Omg, I'm sorry. I viewed it and got lost in other things.

I will add it to my todos and take another look. The biggest problems will be update times. There is a budget and you need live updates. There is no perfect way. You could update through remote notifications or use CloudKit. I recently discovered a nice drop in replacement for AppStorage called cloud storage. Will get back to you

@tzdesign
Copy link

tzdesign commented Dec 4, 2023

@creepymonster it's hard to test, as I just used this app once while checking how my glucose levels are for 2 weeks.

I'm not sure how the sensor is allowed to update, but in general this frequent updates are not possible, except you directly talk to the watch.

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.

4 participants