-
Notifications
You must be signed in to change notification settings - Fork 309
Fix code to also support external apps #546
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
Conversation
|
Here is the summary of changes. You are about to add 1 region tag.
This comment is generated by snippet-bot.
|
| } | ||
| // [END android_compose_drag_and_drop_4] | ||
|
|
||
| val activity = LocalActivity.current |
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.
consider using LocalActivity.current?.let { activity -> so you have a non-null activity
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.
Would it be clear, since it's outside the scope of the snippet?
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 think it's okay. If people see you use activity they will know they need to get an activity from somewhere.
No description provided.