-
-
Notifications
You must be signed in to change notification settings - Fork 423
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
feat(ui): option for automatic update checks #5668
Conversation
app/src/main/java/com/github/libretube/ui/dialogs/UpdateAvailableDialog.kt
Show resolved
Hide resolved
fun onCreateDialog( | ||
changelog: String, | ||
releaseUrl: String, | ||
context: Context, | ||
): Dialog { |
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.
Please refactor this so that the arguments are passed as intent extras, basically as it's been before.
This implementation might crash/dismiss the dialog once the theme is changed on certain devices.
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 have used the intent bundle for passing data and have tested well on android 9 and 13 , the lint is asking for SNAKE-CASE in intent data for these two things(changelog and url), should i do it or ignore it because all others are in camelCase
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.
Thank you!
You don't need to fix ktlint next time, I usually ignore it as well and just format the whole code from time to time. |
the blunders of previous pr are deeply regretted @Bnyro . those should not be existing in first place.