Skip to content

[PB-5926]: show Internxt Drive in SAF picker per session state#430

Open
terrerox wants to merge 3 commits intofeature/PB-5925-kotlin-api-clientfrom
feature/PB-5926-document-query-roots
Open

[PB-5926]: show Internxt Drive in SAF picker per session state#430
terrerox wants to merge 3 commits intofeature/PB-5925-kotlin-api-clientfrom
feature/PB-5926-document-query-roots

Conversation

@terrerox
Copy link
Copy Markdown

InternxtDocumentsProvider.queryRoots() now reads credentials from EncryptedSharedPreferences via a new InternxtAuthManager and returns an empty cursor when no user is signed in, so "Internxt Drive" only appears in the system file picker while the app has an active session; credentials are written by a new RN bridge (InternxtAuthCredentialsModule) from signInThunk and refreshTokensThunk and cleared from signOutThunk, with both paths firing contentResolver.notifyChange(rootsUri) so the picker refreshes without restarting the app, while queryRoots() itself stays strictly synchronous and local-only to avoid freezing the system picker.

@terrerox terrerox self-assigned this Apr 24, 2026
  InternxtDocumentsProvider.queryRoots() now reads credentials from EncryptedSharedPreferences via a new
   InternxtAuthManager and returns an empty cursor when no user is signed in, so "Internxt Drive" only
  appears in the system file picker while the app has an active session; credentials are written by a
  new RN bridge (InternxtAuthCredentialsModule) from signInThunk and refreshTokensThunk and cleared from
   signOutThunk, with both paths firing contentResolver.notifyChange(rootsUri) so the picker refreshes
  without restarting the app, while queryRoots() itself stays strictly synchronous and local-only to
  avoid freezing the system picker.
@terrerox terrerox force-pushed the feature/PB-5926-document-query-roots branch from 0f04755 to 5fe9130 Compare April 24, 2026 04:20
@sonarqubecloud
Copy link
Copy Markdown

@terrerox terrerox requested a review from CandelR April 24, 2026 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant