-
-
Notifications
You must be signed in to change notification settings - Fork 774
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 badge delegate #5138
Update badge delegate #5138
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.
It's not clearing the badge state for me.
Other than just calling this upon clicking, we have logic in place here to verify if the user has seen the first slide.
Perhaps in this situation, you need to create a separate check for logged out users that clears this state after just clicking the item in menu.
We still need the current logic in place to logged-in users, so a different check would work better in this case.
edit: You don't need another delegate method, but work on YiRDataController whether you should clear the badge immediately or still check if the user saw the first slide
WMFComponents/Sources/WMFComponents/Components/Profile/WMFProfileViewModel.swift
Show resolved
Hide resolved
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.
looks good!
Phabricator:
https://phabricator.wikimedia.org/T382932
Notes
Test Steps
Screenshots/Videos