Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Re-fetching of devices certificates on comming back from DeviceDetails [WPB-6970] 🍒 🍒 #3118

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 24, 2024

BugWPB-6970 [Android] Verified device still shows red shield on devices list

This PR was automatically cherry-picked based on the following PR:

Original PR description:


This PR was automatically cherry-picked based on the following PR:

Original PR description:


What's new in this PR?

Issues

When user generate the E2EI certificate in DeviceDetails screen and go back to SelfDevices screen, then current device is still displayed as not-verified.

Causes (Optional)

DevicesCertificates list is not observable, but can be just get (CC doesn't provide any option to observe it).

Solutions

Observe LifecycleEvent of SelfDevicesScreen (added rememberLifecycleEvent() which can be used in LaunchedEffect) and refresh E2EI Certificates on every Resume

@github-actions github-actions bot added cherry-pick PR is cherry-picking changes from another banch size/S labels Jun 24, 2024
@yamilmedina yamilmedina requested review from a team, typfel, alexandreferris, MohamadJaara, vitorhugods and ohassine and removed request for a team June 24, 2024 08:54
…n-comming-back-from-device-details-cherry-pick-cherry-pick
Copy link

@borichellow borichellow enabled auto-merge June 24, 2024 09:33
@borichellow borichellow added this pull request to the merge queue Jun 24, 2024
Copy link
Contributor Author

Built wire-android-staging-release-pr-3118.apk is available for download

Copy link
Contributor Author

Built wire-android-dev-debug-pr-3118.apk is available for download

Merged via the queue into develop with commit fc3ed43 Jun 24, 2024
10 checks passed
@borichellow borichellow deleted the fix/re-fetching-of-devices-certificates-on-comming-back-from-device-details-cherry-pick-cherry-pick branch June 24, 2024 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick PR is cherry-picking changes from another banch size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants