Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[iOS] Fix disable and enable RefreshView #10917

Merged
merged 2 commits into from Jun 4, 2020
Merged

[iOS] Fix disable and enable RefreshView #10917

merged 2 commits into from Jun 4, 2020

Conversation

rmarinho
Copy link
Member

@rmarinho rmarinho commented Jun 1, 2020

Description of Change

We were removing the RefreshControl from the SuperView but not clearing the RefreshControl property on the UIScrollView,

Issues Resolved

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

None

Before/After Screenshots

Not applicable

Testing Procedure

RefreshView should work after disable and enable it on ScrollView.
Gto to Issue 10699
Pull to refresh
Uncheck and check the checkox (this will toggle the RefreshView to Disable and Enable)
Try Pull to refresh again

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@rmarinho rmarinho changed the title Fix issue10699 [iOS] Fix disable and enable RefreshView Jun 1, 2020
@rmarinho rmarinho added this to In progress in Sprint 171 via automation Jun 1, 2020
@rmarinho rmarinho moved this from In progress to Ready for Review (Issues) in Sprint 171 Jun 1, 2020
@rmarinho rmarinho moved this from Ready for Review (Issues) to Ready for Review (PRs) in Sprint 171 Jun 1, 2020
@rmarinho rmarinho linked an issue Jun 1, 2020 that may be closed by this pull request
@rmarinho rmarinho requested a review from jsuarezruiz June 2, 2020 17:14
@samhouts samhouts added 4.6.0 regression on 4.6.0 i/regression labels Jun 2, 2020
@samhouts samhouts assigned jsuarezruiz and unassigned PureWeen Jun 2, 2020
@samhouts samhouts removed their request for review June 2, 2020 23:47
Copy link
Contributor

@jsuarezruiz jsuarezruiz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix-10917

@samhouts samhouts requested review from samhouts and removed request for PureWeen June 4, 2020 23:42
@samhouts samhouts merged commit 9093591 into 4.6.0 Jun 4, 2020
Sprint 171 automation moved this from Ready for Review (PRs) to Done Jun 4, 2020
@samhouts samhouts deleted the fix_issue10699 branch June 4, 2020 23:42
@samhouts samhouts added the approved Has two approvals, no pending reviews, and no changes requested label Jun 5, 2020
@samhouts samhouts added this to the 4.6.0 milestone Jun 9, 2020
@samhouts samhouts added this to Done in v4.6.0 Jun 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4.6.0 regression on 4.6.0 a/refreshview 🔄 approved Has two approvals, no pending reviews, and no changes requested i/regression p/iOS 🍎 t/bug 🐛
Projects
No open projects
Sprint 171
  
Done
v4.6.0
  
Done
Development

Successfully merging this pull request may close these issues.

[Bug] RefreshView IsEnabled Binding not updating
4 participants