Skip to content

Resolved conflicts - Fixed issue #812#876

Closed
neslihanturan wants to merge 4 commits intocommons-app:masterfrom
neslihanturan:ftsalamp-master
Closed

Resolved conflicts - Fixed issue #812#876
neslihanturan wants to merge 4 commits intocommons-app:masterfrom
neslihanturan:ftsalamp-master

Conversation

@neslihanturan
Copy link
Collaborator

@neslihanturan neslihanturan commented Sep 12, 2017

Resolved conflicts at #842 . Functionality works as intended, I have already tested it on different devices, but can someone else also test and merge it? Probably I won't be able to merge this since I am contributor.

ftsalamp and others added 4 commits August 10, 2017 23:27
Conflicts:
	app/src/main/java/fr/free/nrw/commons/contributions/ContributionsListFragment.java
	app/src/main/java/fr/free/nrw/commons/upload/ShareActivity.java
	app/src/main/res/values/strings.xml
@codecov-io
Copy link

Codecov Report

Merging #876 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #876      +/-   ##
=========================================
- Coverage    6.91%   6.86%   -0.05%     
=========================================
  Files          95      95              
  Lines        5018    5052      +34     
  Branches      471     476       +5     
=========================================
  Hits          347     347              
- Misses       4644    4678      +34     
  Partials       27      27
Impacted Files Coverage Δ
...fr/free/nrw/commons/settings/SettingsFragment.java 45.94% <ø> (ø) ⬆️
...mmons/contributions/ContributionsListFragment.java 0% <0%> (ø) ⬆️
...java/fr/free/nrw/commons/upload/ShareActivity.java 0% <0%> (ø) ⬆️
...ee/nrw/commons/media/MediaDetailPagerFragment.java 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c8b0d2c...e6bbf51. Read the comment docs.

@misaochan misaochan mentioned this pull request Sep 13, 2017
@psh
Copy link
Collaborator

psh commented Sep 27, 2017

Merged manually after fixing the conflicts.

@psh psh closed this Sep 27, 2017
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.

4 participants