Skip to content

Conversation

Marocco2
Copy link
Contributor

@Marocco2 Marocco2 commented Aug 25, 2025

Adds DocumentsContract.Root.FLAG_LOCAL_ONLY to the list of the flags in order to show up for third-party apps for easier file syncing with local/cloud file server providers

@Marocco2 Marocco2 changed the title (Android) Pretend Dolphin documents as Local onl (Android) Pretend Dolphin documents as Local only Aug 25, 2025
@Marocco2 Marocco2 marked this pull request as ready for review August 25, 2025 15:46
@Marocco2 Marocco2 changed the title (Android) Pretend Dolphin documents as Local only Android: Pretend Dolphin documents as Local only Aug 25, 2025
@JosJuice
Copy link
Member

Makes sense. I wasn't aware that this flag existed.

Could you please:

  • In the PR title, replace "Pretend" with a different word, like "Mark" or "Set". (We're not pretending, it really is local only.)
  • Change the commit title and commit description so they match the PR title and PR description.

Also, I'm curious: Do you have an example of a file manager app that checks for this flag?

@Marocco2 Marocco2 changed the title Android: Pretend Dolphin documents as Local only Android: Set Dolphin documents as Local only Aug 25, 2025
@Marocco2
Copy link
Contributor Author

Also, I'm curious: Do you have an example of a file manager app that checks for this flag?

Most notably Syncthing, Round Sync and RCX will check for local only storage, so basically it was impossible to add Dolphin folders to be synced for external devices for cross progression

Adds `DocumentsContract.Root.FLAG_LOCAL_ONLY` to the list of the flags in order to show up for third-party apps for easier file syncing with local/cloud file server providers
@Marocco2
Copy link
Contributor Author

I'll publish another commit shortly, hold on for now

…lationships.

Adds `DocumentsContract.Root.FLAG_SUPPORTS_IS_CHILD` to the list of the flags in order to show up for third-party apps for easier file syncing with local/cloud file server providers
@Marocco2
Copy link
Contributor Author

Sorry for the delay: there was a flag missing that some of the third party app won't pick up Dolphin DocumentProvider if not specified.
Now it's ready to be merged

@JosJuice JosJuice merged commit ae3be16 into dolphin-emu:master Sep 16, 2025
13 checks passed
@Marocco2 Marocco2 deleted the patch-1 branch September 16, 2025 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants