webdetails / pentahoLanguagePacks

Pentaho Language Pack Installer plugins
Other
32 stars 81 forks source link

[ta_TA] translated for tamil language #134

Closed shivasakthi18 closed 6 years ago

shivasakthi18 commented 7 years ago

Please review and merge it with master.

Thanks in advance for your support!

wingman-pentaho commented 7 years ago

Hello, This is an automated comment from our build system. Since this pull request originated from a user not in the Pentaho organization, we will not proceed with automatic validation of the pull request. However, rest assured that it will be reviewed manually as soon as we get the chance. Thanks for your contribution!

sivasakthic commented 7 years ago

Hi,

How to get in Pentaho organization, I would like to do more source for open source. I got lots of ideas.

Please guide me to be a community contributor by fixing bugs, sorting out issues etc.,.

Thanks, Sivasakthi.

sivasakthic commented 7 years ago

Please some one take validate this and do a pull request. I have a user waiting for this to implement via marketplace

sivasakthic commented 7 years ago

Who is taking care of this repository...! It's been more than a month now. Someone, please post this language to Pentaho.

carlosrusso commented 7 years ago

Hello Sivasakthi,

I noticed you removed the tags in some files, but kept the original english string there. Was this intentional? The installer automatically removes the tags, so there's no need for that step. The tag is meant to help the translators in locating the bits that haven't been translated yet.

shivasakthi18 commented 7 years ago

Hi,

Tamil is most ancient language, in our state we used to make mix of words in common, if we use pure Tamil Words then people will not understand.

Thanks, Sivasakthi.

shivasakthi18 commented 7 years ago

Hi Carlos,

Let me get feed back on this conversion and proceed further based on input I get.

Thanks, Sivasakthi.

shivasakthi18 commented 7 years ago

Thanks for information on tags, further I Will make it with tags.

carlosrusso commented 7 years ago

It's probably a simple matter of reverting some changes on the files that were not translated to Tamil.

shivasakthi18 commented 7 years ago

I have very little idea on Pentaho code check-ins,

Can you please guide me more, Do you want me to do something like code rever.?

I just did all this with the text editor. Notepad++

carlosrusso commented 6 years ago

I reverted the changes on the files that eliminated the <TRANSLATE ME> tags and merged this PR manually. Closing the PR

sivasakthic commented 6 years ago

Thank you very much.

On Mon, Nov 20, 2017 at 2:05 PM, Carlos Russo notifications@github.com wrote:

I reverted the changes on the files that eliminated the tags and merged this PR manually. Closing the PR

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/webdetails/pentahoLanguagePacks/pull/134#issuecomment-345625558, or mute the thread https://github.com/notifications/unsubscribe-auth/APkYCiSDdBpmEBLUc-MIRP7o7z_eLJTbks5s4TnZgaJpZM4M_OsR .

carlosrusso commented 6 years ago

Please try the build at https://github.com/webdetails/pentahoLanguagePacks/releases/tag/8.0-17.11.17-RC3 If it works, we can publish it in the marketplace. The build seems to work on 7.1 and 8.0, there's a good chance it will work in 7.0 as well.

shivasakthi18 commented 6 years ago

Yes, it works. Please proceed marketplace push.

shivasakthi18 commented 6 years ago

Please let me know whether it is merged to main..?

carlosrusso commented 6 years ago

This is already merged, and is available in the marketplace for versions 7.0 and above.

sivasakthic commented 6 years ago

Perfect, thanks. I updated git Readme file as well.

On Monday, December 18, 2017, Carlos Russo notifications@github.com wrote:

This is already merged, and is available in the marketplace for versions 7.0 and above.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/webdetails/pentahoLanguagePacks/pull/134#issuecomment-352436573, or mute the thread https://github.com/notifications/unsubscribe-auth/APkYCtpugwFYkHRXOqZq43qGTjE6LSKKks5tBnMhgaJpZM4M_OsR .