-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[NT-573] Pull to refresh functionality on the Manage/View Pledge screen #956
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, minor optional suggestions!
self.viewModel.outputs.endRefreshing | ||
.observeForUI() | ||
.observeValues { [weak self] in | ||
self?.rootScrollView.refreshControl?.endRefreshing() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this could reference self.refreshControl
directly π€·ββ - or seeing as we don't customize it and are able to refer to it through the scrollView we could just set it on the scrollView and not have a property referencing it. Up to you.
@@ -294,6 +311,10 @@ final class ManagePledgeViewController: UIViewController, MessageBannerViewContr | |||
self.viewModel.inputs.menuButtonTapped() | |||
} | |||
|
|||
@objc private func handleRefreshControl() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could just be refresh()
or beginRefresh()
π² What
Adds a
UIRefreshControl
to support pull-to-refresh functionality on the Manage/View pledge screen.π€ Why
So that users have a way to refresh the screen and get the most up to date information regarding their backing.
π How
UIRefreshControl
to the root scroll view.valueChanged
action on theUIRefreshControl
to fetching an updated project in the view modelπ See
π Performance
N/A
β Acceptance criteria