jellyfin / jellyfin-androidtv

Android TV Client for Jellyfin
https://jellyfin.org
GNU General Public License v2.0
2.9k stars 493 forks source link

hardcoded texts #507

Closed tinect closed 4 years ago

tinect commented 4 years ago

Describe the bug There is a few hardcoded text for dialogues in code. Please create keys for translations. Here are the one I found within "setmessage". Maybe you find some others in usages of "Utils.showToast" or "Toast.makeText"

To Reproduce

Expected behavior

Logs

Screenshots

System (please complete the following information):

Additional context

nielsvanvelzen commented 4 years ago

FullDetailsActivity.java will be removed soon (replaced by our new details screen) so if anyone likes to work on this issue, ignore that particular file.

PalAditya commented 4 years ago

Hi, it seems this issue is still open, so could I work on it? Also, would it involve changing only the files mentioned in the issue or the entire codebase?

nielsvanvelzen commented 4 years ago

Hi @PalAditya, feel free to work on this issue! The files mentioned in the post are a good start but the codebase did change since this issue was created so there might be more places with hardcoded strings. I think that Android Studio has tools to help find those.