Skip to content

Conversation

hypest
Copy link
Contributor

@hypest hypest commented Apr 6, 2020

This PR "replays" the Dark Mode feature merge trying to have the individual commits be visible in git history.

Looks like the recent merge of the feature (see the 4585d19 commit in #11572) managed to add the feature as a single commit, not correctly attributing the commits to the original authors.

The feature is already merged and no new testing is needed, nor a need to include it to the 14.6 release.

Changes

  1. Reverts 4585d19
  2. Re-merges gutenberg/integrate_release_1.25.0-with-dark-mode from e9ee593

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

malinajirka and others added 30 commits February 10, 2020 12:33
…web-preview

Issue/10648 material web preview
…id into feature/material-theme

# Conflicts:
#	WordPress/src/main/java/org/wordpress/android/ui/reader/subfilter/viewholders/SubfilterListItemViewHolder.kt
#	WordPress/src/main/res/layout/filtered_list_component.xml
…WordPress-Android into issue/10650-material-me-screen

# Conflicts:
#	WordPress/src/main/java/org/wordpress/android/ui/main/MeFragment.java
…WordPress-Android into issue/10649-material-reader-post

# Conflicts:
#	WordPress/src/main/res/values/dimens.xml
…WordPress-Android into issue/10646-material-login-flow

# Conflicts:
#	WordPress/src/main/res/values/styles.xml
…reader-post

Issue/10649 material reader post
…WordPress-Android into issue/fix-lifecycle-crash-with-appcompat-1-1-0
…notifications-list

Issue/10651 material notifications list and settings
…WordPress-Android into issue/10651-material-notification-details
…id into feature/material-theme

# Conflicts:
#	WordPress/src/main/res/layout/reader_listitem_blog.xml
…WordPress-Android into feature/material-theme
…me-screen

Issue/10650 material me screen
@hypest hypest added this to the 14.7 milestone Apr 6, 2020
@peril-wordpress-mobile
Copy link

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

@hypest hypest requested review from khaykov and marecar3 April 6, 2020 07:46
@hypest hypest marked this pull request as ready for review April 6, 2020 07:47
@peril-wordpress-mobile
Copy link

You can test the changes on this Pull Request by downloading the APK here.

@hypest
Copy link
Contributor Author

hypest commented Apr 6, 2020

Just to be sure there's no side effect to the 14.6 WPAndroid release, let's only merge this after v14.6 has been cut.

Copy link
Contributor

@khaykov khaykov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's +0 -0 so looks good to me :)

@marecar3 marecar3 merged commit 42b0918 into develop Apr 9, 2020
@marecar3 marecar3 deleted the replay-dark-mode-commits branch April 9, 2020 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants