Skip to content
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

[TIMOB-26370] Android: "Failed to create external storage directory" when download image and save to camera roll #10443

Merged
merged 2 commits into from Nov 19, 2018

Conversation

ypbnv
Copy link
Contributor

@ypbnv ypbnv commented Nov 8, 2018

JIRA: https://jira.appcelerator.org/browse/TIMOB-26370

Description:
WRITE_EXTERNAL_STORAGE added in Media module where only READ_EXTERNAL_STORAGE is requested. This is in order to make up for a fix in Android 8 which no more grants the whole permission group when a single one is requested.

No unit tests for this PR.

Test case:
There is a good test code in the JIRA ticket.

@build
Copy link
Contributor

build commented Nov 8, 2018

Messages
📖

💾 Here's the generated SDK zipfile.

Generated by 🚫 dangerJS

Copy link
Contributor

@garymathews garymathews left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@jquick-axway jquick-axway left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CR: Pass

@keerthi1032
Copy link
Contributor

FR Passed. Image saved
Test Environment:
Name = Mac OS X
Version = 10.13.6
Architecture = 64bit
Memory = 17179869184
Node.js
Node.js Version = 8.12.0
npm Version = 6.4.1
Titanium CLI
CLI Version = 5.1.1
Titanium SDK
SDK Version = local sdk 8.0.0
Device =pixel android 9,samsung s5 android 6
Emulator = pixel android 8

@keerthi1032 keerthi1032 merged commit a553322 into tidev:master Nov 19, 2018
@keerthi1032
Copy link
Contributor

PR Merged

vijaysingh-axway pushed a commit to vijaysingh-axway/titanium_mobile that referenced this pull request Nov 22, 2018
build pushed a commit to hansemannn/titanium_mobile that referenced this pull request Dec 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants