Skip to content
This repository was archived by the owner on May 15, 2024. It is now read-only.

[Bug][Android] File Picker exception when using third-party file managers#1446

Merged
mattleibow merged 14 commits intoxamarin:mainfrom
dimonovdd:featureFilePicker
Nov 7, 2020
Merged

[Bug][Android] File Picker exception when using third-party file managers#1446
mattleibow merged 14 commits intoxamarin:mainfrom
dimonovdd:featureFilePicker

Conversation

@dimonovdd
Copy link
Copy Markdown
Contributor

@dimonovdd dimonovdd commented Oct 13, 2020

Description of Change

To resolve the exception, you should add this permission:
<uses-permission android:name="android.permission.MANAGE_DOCUMENTS" />

PR corrected as described on #1444 (comment)

Bugs Fixed

Fixes #1444

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Has samples (if omitted, state reason in description)
  • Rebased on top of main at time of PR
  • Changes adhere to coding standard

@jamesmontemagno
Copy link
Copy Markdown
Collaborator

Looks good, appreciate your contribution to the docs as well :)

@jamesmontemagno jamesmontemagno self-requested a review October 15, 2020 01:46
@dimonovdd
Copy link
Copy Markdown
Contributor Author

@jamesmontemagno
have you read this discussion?
#1444

* main:
  [Enhancement] Share Multiple Files (xamarin#1445)
  [FilePicker] Add support for selecting PDF files (Issue 1404) (xamarin#1412)
  Subset of 1030 pr, easy to review (xamarin#1450)
  Add docs for ContinueUserActivity
  Add ContinueUserActivity impl for Universal App Links
@mattleibow mattleibow added this to the 1.6.0 milestone Nov 5, 2020
@mattleibow
Copy link
Copy Markdown
Member

I think we need to have a look at the comments on the issue before we merge.

@mattleibow
Copy link
Copy Markdown
Member

@dimonovdd should we also be removing the intent.AddFlags(ActivityFlags.GrantPersistableUriPermission); flag?

Looking at the docs, it appears that it just opens up the possibility, but doesn't to anything else.

@dimonovdd
Copy link
Copy Markdown
Contributor Author

removing the intent.AddFlags(ActivityFlags.GrantPersistableUriPermission)

Yes, of course. I didn't notice

@mattleibow mattleibow merged commit 54d1ddb into xamarin:main Nov 7, 2020
@dimonovdd dimonovdd deleted the featureFilePicker branch July 12, 2021 20:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][Android] File Picker exception when using third-party file managers

3 participants