WordPress-Android
WordPress-Android copied to clipboard
[Jetpack Focus] Implement Local User Flags ContentProvider and Resolver
Fixes #17144
To test: TODO
Regression Notes
-
Potential unintended areas of impact None
-
What I did to test those areas of impact (or what existing automated tests I relied on) Manual testing and unit tests
-
What automated tests I added (or what prevented me from doing so)
SharedLoginResolverTest.kt
,JetpackLocalUserFlagsFlagTest.kt
,UserFlagsAnalyticsTrackerTest.kt
andUserFlagsResolverTest.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.
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:
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:
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 👍