Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Cannot upload photos if "Don't keep activities" is ON #9351

Closed
Mugurell opened this issue Jan 7, 2021 · 0 comments
Closed

Cannot upload photos if "Don't keep activities" is ON #9351

Mugurell opened this issue Jan 7, 2021 · 0 comments
Assignees
@Mugurell
Milestone
71.0.0 🐈

Comments

@Mugurell
Copy link
Contributor

Mugurell commented Jan 7, 2021 •
edited by data-sync-user
Loading

Original Fenix issue mozilla-mobile/fenix#2997 :

Steps to reproduce

  1. Go to the device's settings and enable the "Don't keep activities"
  2. Go to https://imgbb.com/
  3. Upload a photo.

Expected behavior

I should be able to load the selected photo.

Actual behavior

Here are 2 videos with the behavior when "Don't keep activities" is:

Device information

  • Android device: Nexus 6P (Android 8.1.0)
  • Fenix version: 1.0.1922 (#11500615)

Note:

Not reproducible on https://postimages.org/

logcatNexus.txt

┆Issue is synchronized with this Jira Task
@Mugurell Mugurell self-assigned this Jan 7, 2021
Mugurell added a commit to Mugurell/android-components that referenced this issue Jan 7, 2021
@Mugurell
For mozilla-mobile#9351 - Get camera image even with "Don't keep acti…
a0f71fb 
…vities"

The previous functionality was already based on saving the camera photo in a
specific file and then us reading it from there but what that file name was was
lost in the case of the activity being destroyed as being the case with "Don't
keep activities".
By making that file name top level it will now be available for the entire
duration of the app.
Mugurell added a commit to Mugurell/android-components that referenced this issue Jan 7, 2021
@Mugurell
For mozilla-mobile#9351 - Get camera image even with "Don't keep acti…
7e29c01 
…vities"

The previous functionality was already based on saving the camera photo in a
specific file and then us reading it from there but what that file name was was
lost in the case of the activity being destroyed as being the case with "Don't
keep activities".
By making that file name top level it will now be available for the entire
duration of the app.
Mugurell added a commit to Mugurell/android-components that referenced this issue Jan 13, 2021
@Mugurell
For mozilla-mobile#9351 - Get camera image even with "Don't keep acti…
9e9b70d 
…vities"

The previous functionality was already based on saving the camera photo in a
specific file and then us reading it from there but what that file name was was
lost in the case of the activity being destroyed as being the case with "Don't
keep activities".
By making that file name top level it will now be available for the entire
duration of the app.
mergify bot pushed a commit that referenced this issue Jan 13, 2021
@Mugurell @mergify
For #9351 - Get camera image even with "Don't keep activities"
00ce4bd 
The previous functionality was already based on saving the camera photo in a
specific file and then us reading it from there but what that file name was was
lost in the case of the activity being destroyed as being the case with "Don't
keep activities".
By making that file name top level it will now be available for the entire
duration of the app.
@Amejia481 Amejia481 added this to the 71.0.0 milestone Jan 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Mugurell Mugurell

Labels
None yet
Projects
None yet
Milestone
71.0.0 🐈
Development

No branches or pull requests
2 participants
@Amejia481 @Mugurell