WordPress-Android icon indicating copy to clipboard operation
WordPress-Android copied to clipboard

[Jetpack Focus] Implement Local User Flags ContentProvider and Resolver

Open RenanLukas opened this issue 2 years ago • 2 comments

Fixes #17144

To test: TODO

Regression Notes

  1. Potential unintended areas of impact None

  2. What I did to test those areas of impact (or what existing automated tests I relied on) Manual testing and unit tests

  3. What automated tests I added (or what prevented me from doing so) SharedLoginResolverTest.kt, JetpackLocalUserFlagsFlagTest.kt, UserFlagsAnalyticsTrackerTest.kt and UserFlagsResolverTest.kt

PR submission checklist:

  • [x] I have completed the Regression Notes.
  • [x] I have considered adding accessibility improvements for my changes.
  • [x] I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

RenanLukas avatar Sep 17 '22 02:09 RenanLukas

You can test the Jetpack changes on this Pull Request by downloading an installable build (jetpack-installable-build-pr17170-db77e5d.apk), or scanning this QR code:

wpmobilebot avatar Sep 20 '22 22:09 wpmobilebot

You can test the WordPress changes on this Pull Request by downloading an installable build (wordpress-installable-build-pr17170-db77e5d.apk), or scanning this QR code:

wpmobilebot avatar Sep 20 '22 22:09 wpmobilebot

Warnings
:warning: PR is not assigned to a milestone.
:warning: PR has more than 300 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by :no_entry_sign: dangerJS

Thanks for the review @develric I've applied your suggestions 👍

RenanLukas avatar Sep 29 '22 00:09 RenanLukas