Skip to content

Don't store primary key on connect (behind RC flag)#7580

Draft
CrisBarreiro wants to merge 1 commit intodevelopfrom
feature/cris/ff-do-not-store-primary-key-on-connect
Draft

Don't store primary key on connect (behind RC flag)#7580
CrisBarreiro wants to merge 1 commit intodevelopfrom
feature/cris/ff-do-not-store-primary-key-on-connect

Conversation

@CrisBarreiro
Copy link
Copy Markdown
Collaborator

@CrisBarreiro CrisBarreiro commented Jan 26, 2026

Task/Issue URL: https://app.asana.com/1/137249556945/project/1212227266948491/task/1212935894619116?focus=true

Description

Steps to test this PR

Note

You'll need 3 devices to test out this PR, but only one of them (device A) needs to be an Android device running the code from this PR

Device A has sync off

  • On device A: Enable "Don't keep activities" under developer settings
  • On device A: Fresh install the app
  • On device B: Open sync settings, go to "Sync With Another Device" > "Copy Text Code"
  • On device A: Open sync settings, go to "Sync With Another Device" > "Manually Enter Code" and paste the code from device B
  • On device A, complete sync process and check that when back to "Sync & Backup" screen, synced devices are displayed correctly

Device A has already synced to another device

  • On device A: Enable "Don't keep activities" under developer settings
  • On device A: Fresh install the app
  • On device A: Sync with device C using QR code (not manually entering code)
  • On device A, complete sync process and check that when back to "Sync & Backup" screen, synced devices are displayed correctly
  • On device B: Open sync settings, go to "Sync With Another Device" > "Copy Text Code"
  • On device A: Open sync settings, go to "Sync With Another Device" > "Manually Enter Code" and paste the code from device B
  • On device A, complete sync process and check that when back to "Sync & Backup" screen, synced devices are displayed correctly

Device A has sync off - RC flag disabled

  • On device A: Enable "Don't keep activities" under developer settings
  • On device A: Fresh install the app
  • On device A: Go to feature flag inventory and disable "storePublicKeyWhenGeneratingQRCode" under "sync"
  • On device B: Open sync settings, go to "Sync With Another Device" > "Copy Text Code"
  • On device A: Open sync settings, go to "Sync With Another Device" > "Manually Enter Code" and paste the code from device B
  • On device A, complete sync process and check that when back to "Sync & Backup" screen, synced devices are displayed correctly

UI changes

Before After
!(Upload before screenshot) (Upload after screenshot)

Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

it.checkResult("Creating ConnectQR code failed")
}
}.getOrElse { return it.asErrorResult().alsoFireAccountErrorPixel().copy(code = GENERIC_ERROR.code) }
override suspend fun getConnectQR(): Result<AuthCode> {
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

All production consumers were already on a coroutine context, so I thought it was better to make this function suspend than to launch a new coroutine (or guard against main thread access) when checking the RC flag

Copy link
Copy Markdown
Contributor

@cmonfortep cmonfortep left a comment

Choose a reason for hiding this comment

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

Left comments in Asana.

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.

2 participants