Skip to content

Issue/remove deprecated gif picker#14727

Merged
planarvoid merged 4 commits intodevelopfrom
issue/remove-deprecated-gif-picker
May 28, 2021
Merged

Issue/remove deprecated gif picker#14727
planarvoid merged 4 commits intodevelopfrom
issue/remove-deprecated-gif-picker

Conversation

@khaykov
Copy link
Copy Markdown
Contributor

@khaykov khaykov commented May 26, 2021

This PR removes deprecated GifPickerActivity and all the related classes.

We were using a KEY_SAVED_MEDIA_MODEL_LOCAL_IDS constant from GifPickerActivity, which I extracted into MediaPickerConstants.

To test:

  • GifPickerActivity was not in use, so there is not much to test.
  • Make sure the app builds and installs.

PR submission checklist:

  • I have completed the Regression Notes.
  • 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.

@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented May 26, 2021

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

@khaykov khaykov added this to the 17.5 milestone May 26, 2021
@peril-wordpress-mobile
Copy link
Copy Markdown

Warnings
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS

@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented May 26, 2021

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

@khaykov khaykov marked this pull request as ready for review May 26, 2021 19:50
@khaykov khaykov requested review from develric and planarvoid May 26, 2021 19:50
Copy link
Copy Markdown
Contributor

@planarvoid planarvoid left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@planarvoid planarvoid merged commit 9b3cf5a into develop May 28, 2021
@planarvoid planarvoid deleted the issue/remove-deprecated-gif-picker branch May 28, 2021 12:28
ParaskP7 added a commit that referenced this pull request Dec 5, 2022
This dependency seems that was lastly used only for the no longer
existing 'TenorProviderTest' test class
(see 47ac89a), which was added as part
of this #11587 PR.

However, as part of this #14727 PR, this 'TenorProviderTest' test class
got removed (see f29f6d0).

As such, this leftover from back then can be now safely removed.

PS: And with this change, the corresponding 'androidxTestCoreVersion'
version is no longer necessary for unit test. As such, it is being moved
from the more inclusive 'test' section to the more specific
'android test' section.
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.

2 participants