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] 🍒 #3105

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 18, 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:


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

Copy link

sonarcloud bot commented Jun 18, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor Author

Test Results

930 tests   930 ✅  11m 58s ⏱️
123 suites    0 💤
123 files      0 ❌

Results for commit cd11f18.

Copy link
Contributor Author

APKs built during tests are available here. Scroll down to Artifacts!

@yamilmedina yamilmedina merged commit a669787 into release/candidate Jun 24, 2024
12 checks passed
@yamilmedina yamilmedina deleted the fix/re-fetching-of-devices-certificates-on-comming-back-from-device-details-cherry-pick branch June 24, 2024 08:51
github-actions bot added a commit that referenced this pull request Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants