Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Migrate away from Fragment.startActivityForResult() #19919

Closed
rocketsroger opened this issue Jun 9, 2021 · 5 comments · Fixed by #28351
Closed

Migrate away from Fragment.startActivityForResult() #19919

rocketsroger opened this issue Jun 9, 2021 · 5 comments · Fixed by #28351
Assignees
@t-p-white
Labels
eng:qa:verified QA Verified needs:triage Issue needs triage pin Issues, features, improvements that are still valid
Milestone
111

Comments

@rocketsroger
Copy link
Contributor

rocketsroger commented Jun 9, 2021 •
edited by data-sync-user

Fragment.startActivityForResult(). This API is deprecated in androidx.fragment:fragment 1.3+. Migrate away from it.
https://developer.android.com/training/basics/intents/result

┆Issue is synchronized with this Jira Task
@github-actions github-actions bot added the needs:triage Issue needs triage label Jun 9, 2021
@rocketsroger rocketsroger mentioned this issue Jun 9, 2021
Closed
@stale
Copy link

stale bot commented Dec 6, 2021

See: #17373 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Dec 6, 2021
@gabrielluong gabrielluong added pin Issues, features, improvements that are still valid and removed wontfix labels Dec 8, 2021
@rocketsroger rocketsroger mentioned this issue Feb 15, 2022
Merged
3 tasks
@t-p-white t-p-white self-assigned this Nov 18, 2022
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Dec 5, 2022
@rocketsroger
Close mozilla-mobile#19919: Migrate away from startActivityForResult
a91d2b3
@github-actions github-actions bot mentioned this issue Dec 5, 2022
Closed
4 tasks
@github-actions github-actions bot added the eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged label Dec 5, 2022
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Dec 29, 2022
@rocketsroger
Close mozilla-mobile#19919: Migrate away from startActivityForResult
812939f
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Dec 29, 2022
@rocketsroger
Close mozilla-mobile#19919: Migrate away from startActivityForResult
53ac6ec
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Dec 30, 2022
@rocketsroger
Close mozilla-mobile#19919: Migrate away from startActivityForResult
8a769cf
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Dec 30, 2022
@rocketsroger
Close mozilla-mobile#19919: Migrate away from startActivityForResult
001f8bd
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Dec 30, 2022
@rocketsroger
Close mozilla-mobile#19919: Migrate away from startActivityForResult
6d013f2
@github-actions github-actions bot mentioned this issue Dec 30, 2022
Merged
4 tasks
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Jan 4, 2023
@rocketsroger
Close mozilla-mobile#19919: Migrate away from startActivityForResult
d0e28b0
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Jan 24, 2023
@rocketsroger
Close mozilla-mobile#19919: Migrate away from startActivityForResult
b3b03f9
@mergify mergify bot closed this as completed in #28351 Jan 24, 2023
mergify bot pushed a commit that referenced this issue Jan 24, 2023
@rocketsroger @mergify
Close #19919: Migrate away from startActivityForResult
1e46760
@github-actions github-actions bot reopened this Jan 24, 2023
@github-actions github-actions bot added eng:qa:needed QA Needed and removed eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged labels Jan 24, 2023
@github-actions github-actions bot added this to the 111 milestone Jan 24, 2023
@rocketsroger
Copy link
Contributor Author

For QA, please verify that WebAuthn is still working. Thank you.

@delia-pop
Copy link

Tested on latest Nightly from 01/27 with Google Pixel 6 (Android 13). No issues encountered.

@delia-pop delia-pop added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Jan 27, 2023
@mcarare
Copy link
Contributor

mcarare commented Jan 31, 2023

Reopening this after #28713

@mcarare
Copy link
Contributor

mcarare commented Jan 31, 2023

Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1814096

Change performed by the Move to Bugzilla add-on.

akliuxingyuan pushed a commit to akliuxingyuan/iceraven-browser that referenced this issue Mar 15, 2023
@rocketsroger @mergify
[fenix] Close mozilla-mobile#19919: Migrate away from startActivityFo…
fa7e89d 
…rResult
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@t-p-white t-p-white

Labels
eng:qa:verified QA Verified needs:triage Issue needs triage pin Issues, features, improvements that are still valid
Projects
None yet
Milestone
111
5 participants
@gabrielluong @t-p-white @rocketsroger @mcarare @delia-pop