Skip to content

feat(sync): show linked devices first on the Sync devices page - #1661

Open
A-EDev wants to merge 1 commit into
mainfrom
feat/sync2-linked-default
Open

A-EDev wants to merge 1 commit into
mainfrom
feat/sync2-linked-default

Conversation

@A-EDev

@A-EDev A-EDev commented Nov 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

Phase 7.4 of FLOW-SYNC/2: linked devices become the default way to sync. On the Sync devices page's first step, the Linked devices section now comes first. The FLOW-SYNC/1 send and receive cards move below a divider, under a "One-time transfer" heading. The send and receive steps themselves don't change.

Copy: where the meaning changed, this PR adds new keys so other locales don't keep the old wording (only values/strings.xml is touched; the old keys are removed there):

  • Settings entry subtitle: sync_devices_subtitle ("...over local Wi-Fi") is replaced by sync_devices_summary.
  • Linked devices intro: linked_intro is replaced by linked_always_intro (always in sync, end-to-end encrypted).
  • One-time section: sync_intro is replaced by sync_one_time_title and sync_one_time_body.

sync_devices_title and linked_title keep their text. Nightly note added.

Merge only after the owner's security audit (owner's decision).

Related issue

None.

Change type

  • Bug fix
  • Feature
  • Refactor or maintenance
  • Build, packaging, or CI
  • Documentation

Validation

  • ./gradlew :app:assembleGithubDebug: not run separately; the unit test task below compiled the github debug sources.
  • ./gradlew --offline -Dorg.gradle.jvmargs="..." :app:testGithubDebugUnitTest --tests 'io.github.aedev.flow.sync2.*' --tests 'io.github.aedev.flow.ui.screens.sync.*' --tests 'io.github.aedev.flow.ui.screens.settings.*': 281 tests, 0 failures. --tests 'io.github.aedev.flow.TranslationFormatStringsTest': passed.
  • ./gradlew --offline ... ktlintCheck: exit 0.
  • python3 .github/scripts/nightly_notes.py --check: 75 nightly notes are valid.
  • I manually tested the affected behavior on an Android device or emulator: not yet; part of the Phase 7 device test round.

Test device and Android version: none yet.

Risk and compatibility

UI order and copy only; no sync behaviour, storage or permission change. Removing three English keys leaves their translations orphaned until Weblate drops them.

  • The change does not introduce secrets, private data, or unexpected telemetry.
  • New user-facing text uses Android string resources.
  • Dependency and lockfile changes are intentional and limited to this PR.
  • Room schema changes include the required version bump and migration, or this PR does not change the Room schema.
  • Breaking changes and upgrade steps are clearly documented.

Copilot AI balanced review requested due to automatic review settings November 2, 2026 22:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@github-actions github-actions Bot added documentation Improvements or additions to documentation ui Compose UI, theming, or Material 3 surfaces labels Nov 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation ui Compose UI, theming, or Material 3 surfaces

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants