wordpress-mobile / WordPress-iOS

WordPress for iOS - Official repository
http://ios.wordpress.org/
GNU General Public License v2.0
3.7k stars 1.12k forks source link

Move the Application Passwords entry to the current user details #23814

Closed crazytonyli closed 1 week ago

crazytonyli commented 1 week ago

Two main changes in this PR:

  1. Move self-hosted sites user management code from WordPressUI to the app target.
  2. Move the "Application Passwords" entry from the blog details to the current user details screen.

The application password UI probably needs an update, to fix better with the user details UI. I'll create a separate PR for that.

Regression Notes

  1. Potential unintended areas of impact

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

  3. What automated tests I added (or what prevented me from doing so)

PR submission checklist:

Testing checklist:

wpmobilebot commented 1 week ago
WordPress Alpha📲 You can test the changes from this Pull Request in WordPress Alpha by scanning the QR code below to install the corresponding build.
App NameWordPress Alpha WordPress Alpha
ConfigurationRelease-Alpha
Build Numberpr23814-84ddf5c
Version25.4.2
Bundle IDorg.wordpress.alpha
Commit84ddf5ce9063b387a710d518bf56c2e6b6750777
App Center BuildWPiOS - One-Offs #11055
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.
wpmobilebot commented 1 week ago
Jetpack Alpha📲 You can test the changes from this Pull Request in Jetpack Alpha by scanning the QR code below to install the corresponding build.
App NameJetpack Alpha Jetpack Alpha
ConfigurationRelease-Alpha
Build Numberpr23814-84ddf5c
Version25.4.2
Bundle IDcom.jetpack.alpha
Commit84ddf5ce9063b387a710d518bf56c2e6b6750777
App Center Buildjetpack-installable-builds #10095
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.
dangermattic commented 1 week ago
2 Warnings
:warning: View files have been modified, but no screenshot or video is included in the pull request. Consider adding some for clarity.
:warning: This PR is assigned to the milestone 25.6. The due date for this milestone has already passed.
Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by :no_entry_sign: Danger