profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/zacwest/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

home-assistant/iOS 828

:iphone: Home Assistant for Apple platforms

zacwest/ZSWTappableLabel 153

UILabel subclass for links which are tappable, long-pressable, 3D Touchable, and VoiceOverable.

zacwest/ZSWTaggedString 99

Converts Strings into NSAttributedStrings using an HTML-like markup language.

zacwest/ZSWSuffixTextView 50

UITextView subclass which displays a suffix after editable text and a placeholder.

home-assistant/HAKit 8

Swift library for communicating with Home Assistant.

zacwest/meeting-rooms 6

Simple location-focused calendar display

dropbox/kondo 5

A tool to automate code refactors and modularization

zacwest/ZSWRoundedImage 5

Creates rounded-rectangle UIImages.

zacwest/iosfontsizes.com 4

A simple website that shows the default font sizes on iOS

zacwest/ZSWHierarchicalResultsController 2

Replacement for NSFetchedResultsController. Instead of supporting a single array of objects this exposes one section per object.

push eventhome-assistant/iOS

Zac West

commit sha 8041b997b395edad6479a26dcc8c9205e4d9c7f9

Only signal focus sensor updates in the foreground (#1859) ## Summary Fixes unnecessarily doing a sensor update pass when launching in the background, as well as an occasional crash talking to UIApplication's applicationState off the main thread. ## Screenshots <!-- If this is a user-facing change not in the frontend, please include screenshots in light and dark mode. --> ## Link to pull request in Documentation repository <!-- Pull requests that add, change or remove functionality must have a corresponding pull request in the Companion App Documentation repository (https://github.com/home-assistant/companion.home-assistant). Please add the number of this pull request after the "#" --> Documentation: home-assistant/companion.home-assistant# ## Any other notes <!-- If there is any other information of note, like if this Pull Request is part of a bigger change, please include it here. -->

view details

Home Assistant Bot

commit sha 8bbc216ffb6682c1f0859a9b51ddad1ba431f9ec

Update Localized Strings

view details

push time in 7 hours

push eventhome-assistant/iOS

Zac West

commit sha 8041b997b395edad6479a26dcc8c9205e4d9c7f9

Only signal focus sensor updates in the foreground (#1859) ## Summary Fixes unnecessarily doing a sensor update pass when launching in the background, as well as an occasional crash talking to UIApplication's applicationState off the main thread. ## Screenshots <!-- If this is a user-facing change not in the frontend, please include screenshots in light and dark mode. --> ## Link to pull request in Documentation repository <!-- Pull requests that add, change or remove functionality must have a corresponding pull request in the Companion App Documentation repository (https://github.com/home-assistant/companion.home-assistant). Please add the number of this pull request after the "#" --> Documentation: home-assistant/companion.home-assistant# ## Any other notes <!-- If there is any other information of note, like if this Pull Request is part of a bigger change, please include it here. -->

view details

push time in 11 hours

delete branch home-assistant/iOS

delete branch : sensor-update-background

delete time in 11 hours

PR merged home-assistant/iOS

Only signal focus sensor updates in the foreground cla-signed

Summary

Fixes unnecessarily doing a sensor update pass when launching in the background, as well as an occasional crash talking to UIApplication's applicationState off the main thread.

Screenshots

<!-- If this is a user-facing change not in the frontend, please include screenshots in light and dark mode. -->

Link to pull request in Documentation repository

<!-- Pull requests that add, change or remove functionality must have a corresponding pull request in the Companion App Documentation repository (https://github.com/home-assistant/companion.home-assistant). Please add the number of this pull request after the "#" --> Documentation: home-assistant/companion.home-assistant#

Any other notes

<!-- If there is any other information of note, like if this Pull Request is part of a bigger change, please include it here. -->

+54 -10

1 comment

5 changed files

zacwest

pr closed time in 11 hours

push eventhome-assistant/iOS

Zac West

commit sha 30d9f24e86d5189fa77ed80d1677679691153ed6

Only signal focus sensor updates in the foreground

view details

push time in 12 hours

PR opened home-assistant/iOS

Only signal focus sensor updates in the foreground

Summary

Fixes unnecessarily doing a sensor update pass when launching in the background, as well as an occasional crash talking to UIApplication's applicationState off the main thread.

Screenshots

<!-- If this is a user-facing change not in the frontend, please include screenshots in light and dark mode. -->

Link to pull request in Documentation repository

<!-- Pull requests that add, change or remove functionality must have a corresponding pull request in the Companion App Documentation repository (https://github.com/home-assistant/companion.home-assistant). Please add the number of this pull request after the "#" --> Documentation: home-assistant/companion.home-assistant#

Any other notes

<!-- If there is any other information of note, like if this Pull Request is part of a bigger change, please include it here. -->

+54 -10

0 comment

5 changed files

pr created time in 12 hours

create barnchhome-assistant/iOS

branch : sensor-update-background

created branch time in 12 hours

push eventhome-assistant/iOS

Zac West

commit sha e5f2ebea1ef6de7e0598012833857766483903a0

Reduce verbosity of Shortcuts responses (#1855) * Reduce verbosity of Shortcuts responses * Update localized strings after resetting

view details

Home Assistant Bot

commit sha f6e29fbb2170c9a60823dd6381295ff638c73834

Update Localized Strings

view details

push time in a day

PR closed home-assistant/iOS

Update Localized Strings cla-signed

Automatically created by zacwest.

+275 -17

1 comment

28 changed files

github-actions[bot]

pr closed time in a day

push eventhome-assistant/iOS

Zac West

commit sha e5f2ebea1ef6de7e0598012833857766483903a0

Reduce verbosity of Shortcuts responses (#1855) * Reduce verbosity of Shortcuts responses * Update localized strings after resetting

view details

push time in a day

delete branch home-assistant/iOS

delete branch : quiet-siri

delete time in a day

PR merged home-assistant/iOS

Reduce verbosity of Shortcuts responses cla-signed

Fixes #783.

Summary

<!-- Provide a brief summary of the changes you have made and most importantly what they aim to achieve -->

Screenshots

<!-- If this is a user-facing change not in the frontend, please include screenshots in light and dark mode. -->

Link to pull request in Documentation repository

<!-- Pull requests that add, change or remove functionality must have a corresponding pull request in the Companion App Documentation repository (https://github.com/home-assistant/companion.home-assistant). Please add the number of this pull request after the "#" --> Documentation: home-assistant/companion.home-assistant#

Any other notes

<!-- If there is any other information of note, like if this Pull Request is part of a bigger change, please include it here. -->

+2332 -3387

1 comment

57 changed files

zacwest

pr closed time in a day

issue closedhome-assistant/iOS

Siri Responses to Shortcut Actions are too verbose

When using Siri Shortcuts to run services and automations, she responds with a very raw recollection of the actions in the shortcut.

Hey Siri, Open Sesame! Okay, successfully called cover.open_cover.

Would be great if there were a way to provide an override response such as, "Welcome home, opening the garage." instead.

closed time in a day

xilluzionx

push eventhome-assistant/iOS

Zac West

commit sha d97212831db650a56a1385a7558d030cc752ba5a

Update localized strings after resetting

view details

push time in a day

PR opened home-assistant/iOS

Reduce verbosity of Shortcuts responses

Fixes #783.

Summary

<!-- Provide a brief summary of the changes you have made and most importantly what they aim to achieve -->

Screenshots

<!-- If this is a user-facing change not in the frontend, please include screenshots in light and dark mode. -->

Link to pull request in Documentation repository

<!-- Pull requests that add, change or remove functionality must have a corresponding pull request in the Companion App Documentation repository (https://github.com/home-assistant/companion.home-assistant). Please add the number of this pull request after the "#" --> Documentation: home-assistant/companion.home-assistant#

Any other notes

<!-- If there is any other information of note, like if this Pull Request is part of a bigger change, please include it here. -->

+113 -152

0 comment

3 changed files

pr created time in a day

create barnchhome-assistant/iOS

branch : quiet-siri

created branch time in a day

issue closedhome-assistant/iOS

Onboarding does not alert user if permissions were specifically denied

When setting up from scratch, onboarding does not ask for permissions for motion, background or location.

closed time in a day

olbjan

issue commenthome-assistant/iOS

ASWebAuthenticationSession http support

Not sure there's anything we can do here. If Apple breaks things, we could move to WKWebView then, maybe.

robbiet480

comment created time in a day

issue closedhome-assistant/iOS

ASWebAuthenticationSession http support

Using ASWebAuthenticationSession throws this warning in console: ASWebAuthenticationSession support for http scheme is deprecated and will be removed in a future version of iOS. Use https scheme instead.

Just making this issue to note it.

closed time in a day

robbiet480

issue closedhome-assistant/iOS

Error with Eureka when Interoperating with Objective-C

I am currently working on a fork of HomeAssistant trying to integrate Objective C code into my application. Part of this process involves calling Swift code from an Objective C class. To do this, I have to import the generated Objective-C header file that Xcode makes for the Swift code, "HomeAssistant-Swift.h". This produces an error to the affect of the following, "Cannot find interface declaration for 'EurekaFormViewController', superclass of 'HAFormViewController'". How can I either tell Xcode to ignore the Eureka pod's swift interface or fix the error that it is causing?

To Reproduce Create an Objective-C '.m' file. Add this line to the file to import the generated Objective-C header file:

#import "HomeAssistant-Swift.h"

Now run the project and the error should pop up.

closed time in a day

NoahNetex

issue commenthome-assistant/iOS

Internal URL not working in iOS 15

In Location permission, do you have it set to enable "precise location"?

Tomerello

comment created time in a day

push eventhome-assistant/iOS

Zac West

commit sha ba3ee6d14d387b1b5a1f31e5f51c83d79fb4cc96

Handle confirmable local notifications (#1831) Requires home-assistant/core#54947. Refs #1799. ## Summary Advertises as supporting confirm for local pushes and sends confirmation when receiving a local push. ## Any other notes Confirmable notifications means the app is required to acknowledge receipt of a notification to core, otherwise it will be sent over the normal non-local-push route. This should eliminate a class of potential bugs in the app, where notifications fail to deliver via local push for some reason; however, this means that there's a possibility that notifications may be delayed if such a bug is occurring, so it is hopefully maybe still noticeable/annoying enough to get reports that we can trace back to find the root cause.

view details

Home Assistant Bot

commit sha 7189785bb097fcc64523367f7bb628cc1e8d3f47

Update Localized Strings

view details

push time in 2 days

create barnchzacwest/Quick

branch : initialize-create-tests

created branch time in 3 days

pull request commentQuick/Quick

Support single-test-case selective tests in Quick, fix for non-Quick

The Danger line still fails because of the access issue. Otherwise, I'm unsure what's going on in the Xcode 13 CI, but the same tests pass locally:

Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testNonQuickSpecRequestingNoTestCase]' started.
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testNonQuickSpecRequestingNoTestCase]' passed (0.001 seconds).
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testNonQuickSpecRequestingOneTestCase]' started.
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testNonQuickSpecRequestingOneTestCase]' passed (0.001 seconds).
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testQuickSpecRequestingNoTestCase]' started.
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testQuickSpecRequestingNoTestCase]' passed (0.002 seconds).
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testQuickSpecRequestingOneTestCase]' started.
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testQuickSpecRequestingOneTestCase]' passed (0.001 seconds).
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testQuickSpecTestInvocationsForAllTests]' started.
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testQuickSpecTestInvocationsForAllTests]' passed (0.001 seconds).
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testQuickSpecTestInvocationsForSelectedTests]' started.
Test Case '-[Quick___iOSTests.QuickSpec_SelectedTests testQuickSpecTestInvocationsForSelectedTests]' passed (0.001 seconds).

I did have to mutate what XCTest the Nimble framework was pointing to in my local checkout (it's trying to link against either the old location), perhaps there's some underlying oddness going on there.

zacwest

comment created time in 3 days

push eventhome-assistant/iOS

Zac West

commit sha ba3ee6d14d387b1b5a1f31e5f51c83d79fb4cc96

Handle confirmable local notifications (#1831) Requires home-assistant/core#54947. Refs #1799. ## Summary Advertises as supporting confirm for local pushes and sends confirmation when receiving a local push. ## Any other notes Confirmable notifications means the app is required to acknowledge receipt of a notification to core, otherwise it will be sent over the normal non-local-push route. This should eliminate a class of potential bugs in the app, where notifications fail to deliver via local push for some reason; however, this means that there's a possibility that notifications may be delayed if such a bug is occurring, so it is hopefully maybe still noticeable/annoying enough to get reports that we can trace back to find the root cause.

view details

push time in 3 days

delete branch home-assistant/iOS

delete branch : confirm-local-push

delete time in 3 days

PR merged home-assistant/iOS

Handle confirmable local notifications cla-signed

Requires home-assistant/core#54947. Refs #1799.

Summary

Advertises as supporting confirm for local pushes and sends confirmation when receiving a local push.

Any other notes

Confirmable notifications means the app is required to acknowledge receipt of a notification to core, otherwise it will be sent over the normal non-local-push route. This should eliminate a class of potential bugs in the app, where notifications fail to deliver via local push for some reason; however, this means that there's a possibility that notifications may be delayed if such a bug is occurring, so it is hopefully maybe still noticeable/annoying enough to get reports that we can trace back to find the root cause.

+142 -3

1 comment

4 changed files

zacwest

pr closed time in 3 days

push eventhome-assistant/iOS

Home Assistant Bot

commit sha d984ff615626aa739c1193636db7e22732ac86d1

Update Localized Strings

view details

push time in 3 days

pull request commenthome-assistant/core

Allow confirming local push notifications

Tested this using the iOS PR linked above:

  • with support_confirm, a notification that was confirmed was not sent again
  • with support_confirm, a notification that was not confirmed was sent again
  • without support_confirm, a notification that was not confirmed was not sent again
balloob

comment created time in 3 days