Change ActivityResultRegistry invoke() to onLaunch() [157496491]

Fixed

Bug

Status Update

No update yet.

Description

jb...@google.com

created issue #1

May 26, 2020 08:21PM

The invoke() method is the only API exposed from ActivityResultRegistry. This used to make since because ActivityResultLauncher used the kotlin invoke() and invoking the launcher would call invoke() on the registry. Since ActivityResultLauncher has moved to using launch() this connection is no longer clear.

We should change the invoke() in ActivityResultRegistry to be onLaunch() so it is clear that it is executed once the call to launch() is made.

Comments

ap...@google.com <ap...@google.com> #2May 26, 2020 09:33PM

Hi Ed, Thank you so much for these suggestions. I've been reviewing them and merging them in. Hopefully it should be live. I've included a thank you note too in the article.

jb...@google.com <jb...@google.com> #3May 26, 2020 09:36PM

Marked as fixed.

Great! Thanks a lot, I'll look for the live updates soon!

Issue 157496491

Description

Issue summary

Comments

ap...@google.com <ap...@google.com> #2May 26, 2020 09:33PM

jb...@google.com <jb...@google.com> #3May 26, 2020 09:36PM

Add comment

Issue metadata