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

Migrate from WatchKit storyboard to SwiftUI and its lifecycle #739

Open
wants to merge 29 commits into
base: develop
Choose a base branch
from

Conversation

timbms
Copy link
Contributor

@timbms timbms commented Nov 7, 2023

Get rid of a build warning that WatchKit storyboards are deprecated as of watchOS 7.0.
Addresses #738 and #748

timbms and others added 7 commits November 7, 2023 23:42
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
saveTrustedCertificate() writes with PropertyListEncoder instead of NSKeyedArchiver
loadTrustedCertificates() attempts first decoding with PropertyListDecoder() if error is thrown it unarchives with NSKeyedUnarchiver()

Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
…ge from closure to function (in the past this allowed reuse code for challenge responses between Kingfisher and Alamofire libraries), simplified their signature as the first two parameters were ignored, avoids useless initializers for URLSession and URLSessionDataTask (#736)

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Co-authored-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
* Migrate onReceiveSessionTaskChallenge and onReceiveSessionTaskChallenge from closure to function (in the past this allowed reuse code for challenge responses between Kingfisher and Alamofire libraries), simplified their signature as the first two parameters were ignored, avoids useless initializers for URLSession and URLSessionDataTask

* Fix for keyWindow deprecation warning

---------

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Co-authored-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
)

Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
…mpler https://www.swiftbysundell.com/tips/optional-swiftui-views/

Signed-off-by: Tim Müller-Seydlitz <timbms@gmail.com>
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
…able from Swift 5.7

Switched from ScrollView to List
FrameRow without background
Hide .placeholder icon

Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
@digitaldan
Copy link
Contributor

Cool! is this ready for review or are you still working on it?

@timbms
Copy link
Contributor Author

timbms commented Nov 13, 2023

Am still working on it. The sitemap is hard wired. This should be changed.

@timbms
Copy link
Contributor Author

timbms commented Nov 16, 2023

Also considers #722 - allowing to set the sitemap path to, for instance, uicomponents_watch

Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
@timbms timbms linked an issue Dec 27, 2023 that may be closed by this pull request
3 tasks
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
…of challenges and header modifiers

Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Disabling swiftformat because it introduces double header

Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Progress on getting the openHABversion via OpenHABWatchTracker / not yet complete
Getting iconType into watch app / sending still to be implemented

Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Avoid swiftformat on too long line
Rework of OpenHABWatchTracker with NWPathMonitor, able to recognize openHABversion and passing it in particular to Endpoint.icon
New files for class OpenHABImageDownloaderOperation: SDWebImageDownloaderOperation and class OpenHABRequestModifier: SDWebImageDownloaderRequestModifier
Removed Kingfisher as dependency in watch app. Now replaced by SDWebImage

Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecation warning for WatchKit usage in watch app
2 participants