Include WordPress-Login-Flow `1.18.0` with its git history
- [x] Review and merge #21301 first
[!TIP] This PR is a result of
git filter-repofiltering. To read more about the process, please see internal post: paqN3M-m2-p2. The project discussed there is different, but the same rules apply here.
Description
This PR is similar to #21260 - it adds the source code of WordPress-Login-Flow in version 1.18.0 with its git history to the WordPress-Android codebase.
I've also pushed 1_18_0_filtered_for_vendoring branch to WordPress-Login-Flow repository: https://github.com/wordpress-mobile/WordPress-Login-Flow-Android/tree/1_18_0_filtered_for_vendoring
The filter-repo command was as simple as
git filter-repo --path WordPressLoginFlow
Review
There's not much to review here. You can verify that last commit from this PR is the same as last commit from 1.18.0 release (link) in a sense of content and message. Hashes won't be the same, as the history is rewritten.
| 12 Warnings | |
|---|---|
| :warning: | strings.xml files should only be updated on release branches, when the translations are downloaded by our automation. |
| :warning: | This PR is larger than 300 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews. |
| :warning: | Class CreatedAccountSource is missing tests, but unit-tests-exemption label was set to ignore this. |
| :warning: | Class LoginWpcomService is missing tests, but unit-tests-exemption label was set to ignore this. |
| :warning: | Class LoginState is missing tests, but unit-tests-exemption label was set to ignore this. |
| :warning: | Class LoginNotification is missing tests, but unit-tests-exemption label was set to ignore this. |
| :warning: | Class OnCredentialsOK is missing tests, but unit-tests-exemption label was set to ignore this. |
| :warning: | Class TwoFactorRequested is missing tests, but unit-tests-exemption label was set to ignore this. |
| :warning: | Class SiteUtils is missing tests, but unit-tests-exemption label was set to ignore this. |
| :warning: | Class PasskeyRequest is missing tests, but unit-tests-exemption label was set to ignore this. |
| :warning: | Class PasskeyRequestData is missing tests, but unit-tests-exemption label was set to ignore this. |
| :warning: | Class SavedState is missing tests, but unit-tests-exemption label was set to ignore this. |
Generated by :no_entry_sign: Danger
📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
| App Name | WordPress |
|
| Flavor | Jalapeno | |
| Build Type | Debug | |
| Version | pr21266-06504ed | |
| Commit | 06504ed0ee029dc93fc1fb3e3206e085420e5bfe | |
| Direct Download | wordpress-prototype-build-pr21266-06504ed.apk |
📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
| App Name | Jetpack |
|
| Flavor | Jalapeno | |
| Build Type | Debug | |
| Version | pr21266-06504ed | |
| Commit | 06504ed0ee029dc93fc1fb3e3206e085420e5bfe | |
| Direct Download | jetpack-prototype-build-pr21266-06504ed.apk |
Quality Gate passed
Issues
57 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
2.9% Duplication on New Code
Review There's not much to review here. You can verify that last commit from this PR is the same as last commit from 1.18.0 release (link) in a sense of content and message. Hashes won't be the same, as the history is rewritten.
- It is very hard to review this change, so I am just going to trust the process here. 😊
- I tried compare the history of
loginincluded in this PR to that on Login itself, but again, it was very hard for me to conclude that nothing was missing, lots of moving pieces. As such, and as explained above, I can only trust the process. 🤞 - Having said that, I did verify that the last commit in this PR is the same as the commit from 1.18.0. As such, this looks good. 👍
PS: There are these 2 top commits (Bump fluxc to trunk + Attempt to fix build issue), but those aren't WordPressLoginFlow related and as such can be ignored.