Skip to content

Avoid jargon and slang, change for default, de and pl (#2874) #2876

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

Merged

Conversation

marcin-kwasniak
Copy link
Contributor

"check category failure title" and "check category success title" shouldn't use the words "success" and "failure"

Fixes #{#2874} for default, de and pl

@misaochan
Copy link
Member

I don't think we should be editing the non-default language strings directly, everything except default will be overriden by translatewiki's automatic updates AFAIK.

For the default, I'll leave a comment on the code itself.

@marcin-kwasniak
Copy link
Contributor Author

Ok, so i close this pull request.

@misaochan
Copy link
Member

@marcin-kwasniak , you can make changes within the PR itself. If you modify your code, commit it to the same branch and push the branch, we will automatically get your changes.

@codecov-io
Copy link

codecov-io commented Apr 11, 2019

Codecov Report

Merging #2876 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #2876      +/-   ##
=========================================
- Coverage    3.42%   3.42%   -0.01%     
=========================================
  Files         246     246              
  Lines       12193   12208      +15     
  Branches     1071    1072       +1     
=========================================
  Hits          418     418              
- Misses      11744   11759      +15     
  Partials       31      31
Impacted Files Coverage Δ
...fr/free/nrw/commons/settings/SettingsFragment.java 0% <0%> (ø) ⬆️
.../fr/free/nrw/commons/widget/PicOfDayAppWidget.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 6bde4f9...c871413. Read the comment docs.

@marcin-kwasniak
Copy link
Contributor Author

@misaochan I made change, is it ok now?

@misaochan
Copy link
Member

misaochan commented Apr 12, 2019

Hi @marcin-kwasniak ,

My suggestion was to have "Category check successfully requested" for the success string, because "category check requested" does not necessarily mean success IMO. Your text for the failure string was fine, there was no need to remove it.

@marcin-kwasniak
Copy link
Contributor Author

My mistake, new text for fail is commited.

@misaochan
Copy link
Member

Thanks, and congrats on your first PR! :)

@misaochan misaochan merged commit 4f76f1b into commons-app:master Apr 13, 2019
@marcin-kwasniak marcin-kwasniak deleted the avoid-jargon-and-slang-#2874 branch April 14, 2019 09:50
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