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

Update to Xcode 15.0.1 / iOS 17.0.1 #75

Merged
merged 10 commits into from
Oct 26, 2023
Merged

Update to Xcode 15.0.1 / iOS 17.0.1 #75

merged 10 commits into from
Oct 26, 2023

Conversation

caiozullo
Copy link
Member

No description provided.

…r's `viewIsAppearing` lifecycle method

- This change is needed since UIRefreshControl updates must happen when the view is on-screen on iOS17+ (it ignores updates otherwise!).

- `viewDidLoad` is called once per VC, but the new `viewIsAppearing` may be called multiple times - so we added logic to ensure the refresh only happens in the first `viewIsAppearing` occurrence to mimic the previous behavior in `viewDidLoad`.

- When running tests, the views are never added on-screen, so UIRefreshControl doesn't work as expected on iOS 17+. To allow testability, we replace the UIRefreshControl with a FakeUIRefreshControl test-double.
@caiozullo caiozullo changed the title Update to Xcode 15.0 / iOS 17.0 Update to Xcode 15.0.1 / iOS 17.0 Oct 26, 2023
@caiozullo caiozullo force-pushed the xcode15_0 branch 2 times, most recently from 4c0d56f to 0b310f9 Compare October 26, 2023 08:32
@caiozullo caiozullo changed the title Update to Xcode 15.0.1 / iOS 17.0 Update to Xcode 15.0.1 / iOS 17.0.1 Oct 26, 2023
@mapostolakis mapostolakis merged commit 888c9fe into master Oct 26, 2023
@mapostolakis mapostolakis deleted the xcode15_0 branch October 26, 2023 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants