Skip to content

feat(users): migrate UsersDao, UsersRepository and UserManager to coroutines - #6644

Draft
mahibi wants to merge 1 commit into
masterfrom
migrateUserManagerToCoroutines
Draft

feat(users): migrate UsersDao, UsersRepository and UserManager to coroutines#6644
mahibi wants to merge 1 commit into
masterfrom
migrateUserManagerToCoroutines

Conversation

@mahibi

@mahibi mahibi commented Sep 4, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

WIP

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔖 Capability is checked or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@mahibi mahibi self-assigned this Sep 4, 2026
@mahibi
mahibi force-pushed the migrateUserManagerToCoroutines branch from fb61129 to eed482b Compare September 8, 2026 15:39
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/34246126796/artifacts/10064452703
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@mahibi
mahibi force-pushed the migrateUserManagerToCoroutines branch from eed482b to 08cffb6 Compare September 10, 2026 10:35
@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/34466901916/artifacts/10150441248
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@mahibi
feat(users): migrate UsersDao, UsersRepository and UserManager to cor…
7ffb02c 
…outines

Convert the user data layer (UsersDao, UsersRepository/Impl) from RxJava2
to suspend fun/Flow, mirroring the ConversationsDao pattern already used
elsewhere. UserManager gains suspend counterparts for every method
alongside its existing RxJava-typed API, which is now deprecated and
bridges to the suspend implementation via kotlinx-coroutines-rx2, so the
~50 existing call sites keep compiling unchanged. Follow-up PRs will
migrate those callers in batches and then remove the deprecated methods.

Assisted-by: Claude Code:claude-sonnet-5

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>
@mahibi
mahibi force-pushed the migrateUserManagerToCoroutines branch from 08cffb6 to 7ffb02c Compare September 14, 2026 13:29
@github-actions

github-actions Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/34849556867/artifacts/10350730917
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@mahibi mahibi

Labels

AI assisted

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mahibi