Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed typo in nearbyActivity #1394

Merged
merged 1 commit into from
Apr 16, 2018
Merged

Conversation

97balakrishnan
Copy link
Contributor

Fixed small typo in nearbyActivity :)

@codecov-io
Copy link

codecov-io commented Mar 31, 2018

Codecov Report

Merging #1394 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1394      +/-   ##
=========================================
- Coverage     3.3%   3.27%   -0.04%     
=========================================
  Files         128     128              
  Lines        6837    7145     +308     
  Branches      669     736      +67     
=========================================
+ Hits          226     234       +8     
- Misses       6596    6895     +299     
- Partials       15      16       +1
Impacted Files Coverage Δ
...ava/fr/free/nrw/commons/nearby/NearbyActivity.java 0% <0%> (ø) ⬆️
...fr/free/nrw/commons/settings/SettingsFragment.java 27.5% <0%> (-1.08%) ⬇️
...java/fr/free/nrw/commons/category/CategoryDao.java 0% <0%> (ø) ⬆️
...e/nrw/commons/category/CategorizationFragment.java 0% <0%> (ø) ⬆️
.../fr/free/nrw/commons/nearby/NearbyMapFragment.java 0% <0%> (ø) ⬆️
.../java/fr/free/nrw/commons/nearby/DirectUpload.java 0% <0%> (ø) ⬆️
...n/java/fr/free/nrw/commons/CommonsApplication.java 33.33% <0%> (+2.72%) ⬆️

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 e23f752...baae648. Read the comment docs.

@neslihanturan
Copy link
Collaborator

Thanks @97balakrishnan but I think this is fine as is. Since people used to use that word as Lng:)

@97balakrishnan
Copy link
Contributor Author

It was like LatLang i have changed it to Lng
Since it represents Latitude and Longitude it should be either LatLong or LatLng.. pls go through the changes once again if possible

I dont know if this has been corrected already

@neslihanturan neslihanturan reopened this Apr 16, 2018
@neslihanturan
Copy link
Collaborator

Sorry for misunderstanding, thanks @97balakrishnan

@neslihanturan neslihanturan merged commit f8fb9f1 into commons-app:master Apr 16, 2018
@97balakrishnan
Copy link
Contributor Author

Thank you :)

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.

3 participants