Commit Graph

5 Commits

Author SHA1 Message Date
mat1th
97834bfd5e Update swift lint and format + appy fixes (#2585)
## Summary
Swift lint and swiftformat are outdated. This PR does update those +
applies the new formatting form swiftformat.
There is 1 swift file with a manual change:
`Sources/Vehicle/Templates/Areas/CarPlayAreasViewModel.swift`. This is
done because `swiftlint` did create the following swiftlint error:
`error: Cyclomatic Complexity Violation: Function should have complexity
10 or less; currently complexity is 11 (cyclomatic_complexity)`.

Because it does change a lot of files the question is if we want to
finetune the `swiftformat` rules.

## Screenshots
No user facing changes.

## Link to pull request in Documentation repository
NA

## Any other notes
NA
2024-02-22 13:06:39 +01:00
Zac West
ab56fd7afa Revert "Remove Firebase (#2028)" (#2051)
This reverts commit 2c4bab1d5c.
2022-01-23 18:45:04 +00:00
Zac West
2c4bab1d5c Remove Firebase (#2028)
## Summary
Removes FCM and related SDKs in favor of using the server introduced in #2024.

## Any other notes
This will require testing. It may be worth reverting this PR to introduce a toggle if this lasts more than 1 beta cycle.

Future work includes figuring out the permanent location for these services; they are on fly.io right now but that may cost money, but for now I just want TestFlight builds to use it.
2021-12-26 09:01:29 +00:00
Zac West
b5f4b2fe40 Enable Mac interface idiom & Macify settings (#1600)
Fixes #1305.

## Summary
Enables the new-to-macOS 11 interface idiom of Mac, which removes any interface scaling and brings more Mac-like UI.

## Screenshots
<img width="350" alt="image" src="https://user-images.githubusercontent.com/74188/115946708-ea115500-a477-11eb-9e42-378e76d6e136.png"><img width="350" alt="image" src="https://user-images.githubusercontent.com/74188/115946711-eed60900-a477-11eb-8591-fd01a3711787.png">

## 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
- Updates Settings on Big Sur to use a toolbar to swap between root-level settings screens.
- Fixes blurry web view, only on Big Sur.
- `UISwitch` as a checkbox looks kind of weird; may need to do some Eureka-level changes to make SwitchRow use the `title` property on `UISwitch` to make it connected so the whole thing is clickable. This gets weird because the font size is larger for this property.
2021-04-24 07:08:30 +00:00
Zac West
3be1a97d24 Reshuffle Notifications settings screen (#1475)
## Summary
Moves Categories, Notifications and Rate Limits out of the root of the Notifications settings. Relocates and slims down the debug-related things.

## Screenshots
![Image](https://user-images.githubusercontent.com/74188/107476149-26484100-6b2a-11eb-93e8-7a48952e01c6.png)
![Image 2](https://user-images.githubusercontent.com/74188/107476208-39f3a780-6b2a-11eb-96ea-e57ec1a0a2a5.png)
![Image-2](https://user-images.githubusercontent.com/74188/107476251-50016800-6b2a-11eb-9af7-ddfda3274966.png)
![Image-2 2](https://user-images.githubusercontent.com/74188/107476286-627ba180-6b2a-11eb-9adf-8625bde61105.png)
![Image-2 3](https://user-images.githubusercontent.com/74188/107476329-78896200-6b2a-11eb-97e5-25b5f89f2872.png)

## Link to pull request in Documentation repository
Not a lot of doc changes for this one. They all speak generically enough that the same copy/directions works the same here.
Documentation: home-assistant/companion.home-assistant#461

## Any other notes
- Shows the rate limit's remaining value as its detail label in the row; updates both via pull-to-refresh in the Rate Limits screen.
- Fixes  the notification token being reset not updating the server with the new value immediately.
2021-02-11 07:47:33 +00:00