Siren icon indicating copy to clipboard operation
Siren copied to clipboard

feat(locale): add language support: Traditional Chinese.

Open samuelchou opened this issue 4 years ago • 7 comments

I'm trying to add Chinese(Traditional) support. It should be a easy fix...

samuelchou avatar Dec 02 '20 10:12 samuelchou

@eggheadgames I'm just confused with the CircleCI error. How could I fix that? I'm willing to do the work. 💪

samuelchou avatar Dec 10 '20 09:12 samuelchou

@samuelchou Sorry for the slow response. My guess is that the CircleCi configuration needs to be updated for their version 2. I.e. similar to this change:

https://github.com/eggheadgames/android-about-box/commit/127b89b2c086fb90cd859b74a8862b11be7e0b75#diff-f3e9d251e83fc546f51aa7cfc8ae605d64f20c232423c4b4eaf968f227b09ac6

If you have some time to do that, it is much appreciated!

mikemee avatar Dec 11 '20 03:12 mikemee

Interesting...CircleCI failed at.... uh, unexpectedly.

FAILURE: Build failed with an exception.

* What went wrong:
Could not determine java version from '11.0.9.1'.

If that was what I think, gradle source seems to be out-dated...?

samuelchou avatar Dec 14 '20 02:12 samuelchou

Ok...there are more gradle problems than I expected. I'll try to fix it by the way.

1: Task failed with an exception.
-----------
* What went wrong:
A problem occurred evaluating project ':library'.
> Failed to apply plugin [id 'com.github.dcendents.android-maven']
   > Could not create plugin of type 'AndroidMavenPlugin'.
      > Unable to determine constructor argument #1: missing parameter of type Factory, or no service of type Factory<LoggingManagerInternal>.
2: Task failed with an exception.
-----------
* What went wrong:
A problem occurred configuring project ':library'.
> compileSdkVersion is not specified. Please add it to build.gradle

samuelchou avatar Dec 14 '20 02:12 samuelchou

After I tried updating android-maven , problem shows up in Android Studio:

Caused by: org.gradle.api.plugins.UnknownPluginException: Plugin with id 'findbugs' not found.

And it seems that findbugs is removed, implying that all of the related codes need to be changed: https://stackoverflow.com/a/59102723/9735961

I don't know man, the fix seems like a HUGE work to the code, and it might be inappropriate to be done in this pull request...? Please let me know if you have any thought @mikemee .

samuelchou avatar Dec 14 '20 02:12 samuelchou

Since spotbugs wasn't function correctly, I just removed related codes temporarily to fix other problems. I can add it back if needed. Also, I fix several bugs Circle CI detected. But bugs seem to be endless...

I now feel it's a disaster to upgrade gradle... 😭

samuelchou avatar Dec 14 '20 08:12 samuelchou

@samuelchou Sorry for the hassle. I went through most of this same stuff with the https://github.com/eggheadgames/android-about-box/ when I updated it to work with current versions of Android Studio and new releases of Android. There may be some clues there, as the projects were started around the same time and set up the same way.

Unfortunately, I don't use Siren in my Android apps any longer, so I wasn't forced to update it at the same time.

It looks like you're making steady progress. You might want to check the android-about-box project to see what version of Gradle I used there and also the upgrade PR: https://github.com/eggheadgames/android-about-box/pull/52

mikemee avatar Dec 19 '20 08:12 mikemee