-
Notifications
You must be signed in to change notification settings - Fork 25
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
actions: less warnings is more (fixes #3699) #3703
Conversation
app/src/main/java/org/ole/planet/myplanet/ui/team/MyTeamsDetailFragment.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/ole/planet/myplanet/ui/team/MyTeamsDetailFragment.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/ole/planet/myplanet/ui/team/MyTeamsDetailFragment.kt
Outdated
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.
agree with @Okuro3499. The first comment would simply require an else branch that satisfies that same functionality as "createNews(map, mRealm, user, imageList)"
app/src/main/java/org/ole/planet/myplanet/ui/team/MyTeamsDetailFragment.kt
Show resolved
Hide resolved
app/src/main/java/org/ole/planet/myplanet/ui/team/MyTeamsDetailFragment.kt
Outdated
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.
Other than the really small issues, I think the rest of this pr looks good!
app/src/main/java/org/ole/planet/myplanet/ui/team/MyTeamsDetailFragment.kt
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.
Works for me. I would solve the conflicts that are shown below to make the merge easier, and those should be very minor conflicts.
fixes #3699