Removed butterknife from MediaLicenseFragment #5426
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Not sure why CI is consistently failing to build after only a few seconds. after about 45 seconds |
|
I wanted to run the unit tests on my machine, but I am getting this build error again (any idea what might be happening by the way?). Do the unit tests run fine on your machine? If yes I will merge. :-) |
This is now fixed, would you mind rebasing from main? :-) |
psh
force-pushed
the
psh/remove-butterknife-from-media-license-fragment
branch
from
5d4834e to
dce1621
Compare
|
Rebase done, and things seem to be passing again. Thanks for the fix! 👍 |
nicolas-raoul
approved these changes
Jan 15, 2024
|
Thanks for your pull request! :-) |
ShashwatKedia
pushed a commit
to ShashwatKedia/apps-android-commons
that referenced
this pull request
Jan 15, 2024
nicolas-raoul
added a commit
that referenced
this pull request
Jan 15, 2024
* Resolved issue #4513 * Localisation updates from https://translatewiki.net. * NearbyParentFragment : added referer (#5417) * NearbyParentFragment : added referer In file NearbyParentFragment.java, I added header property, i.e., the referer - http://maps.wikimedia.org/ and set tile source to wikimedia. * Reworded comments --------- Co-authored-by: Nicolas Raoul <[email protected]> * Localisation updates from https://translatewiki.net. * Localisation updates from https://translatewiki.net. * Removed butterknife from the quiz result activity (#5425) * ConnectRefuseError: removed the proxy from properties.gradle file to allow downloading of build tools during CI. (#5434) * NearbyParentFragment : added referer In file NearbyParentFragment.java, I added header property, i.e., the referer - http://maps.wikimedia.org/ and set tile source to wikimedia. * Reworded comments * sdkmanager: added installation command for build-tools-30.0.3 * Revert "sdkmanager: added installation command for build-tools-30.0.3" This reverts commit b3e5019. * Update android.yml * Update gradle.properties * android.yml: removed extra debug commands Removed some debug commands because they are no longer needed. --------- Co-authored-by: Nicolas Raoul <[email protected]> * Removed butterknife and inlined a couple of tiny methods (#5426) * Resolves #2239 by adding a compass arrow for direction of nearest item (#5433) * Resolves issue #2239 by adding an arrow for direction * Removed unnecessary change in styles.xml * spacing * javadoc --------- Co-authored-by: Nicolas Raoul <[email protected]> * Added voice input for caption and description (#5415) * Fixed Grey empty screen at Upload wizard caption step after denying files permission * Empty commit * Fixed loop issue * Created docs for earlier commits * Fixed javadoc * Fixed spaces * Added added basic features to OSM Maps * Added search location feature * Added filter to Open Street Maps * Fixed chipGroup in Open Street Maps * Removed mapBox code * Removed mapBox's code * Reformat code * Reformatted code * Removed rotation feature to map * Removed rotation files and Fixed Marker click problem * Ignored failing tests * Added voice input feature * Fixed test cases * Changed caption and description text --------- Co-authored-by: translatewiki.net <[email protected]> Co-authored-by: Rohit Verma <[email protected]> Co-authored-by: Nicolas Raoul <[email protected]> Co-authored-by: Paul Hawke <[email protected]> Co-authored-by: Kanahia <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Removed butterknife from MediaLicenseFragment and inlined a couple of tiny methods