-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 Lifecycle to 2.5.1. #8683
Conversation
For library bumps, it would be good to include the changelog link. |
667000f
to
0dc6320
Compare
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 added a commit refactoring the view model factory according to what the changelog suggests. I tested and everything seems to work fine.
d109017
to
311d392
Compare
Kudos, SonarCloud Quality Gate passed! |
@Stypox Is there a reason you improved only the FeedViewModel? Could the improvements also be applied to the FeedGroupDialogViewModel? |
No, my bad, sorry |
What is it?
Description of the changes in your PR
Fixes the following issue(s)
APK testing
The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR.
Due diligence