1042973 - [Notes] Add Bengali (bn-BD) Translation to Notes App

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Notes, defect, P2)

Description

[Dylan Oliver [:doliver]]

+++ This bug was initially created as a clone of Bug #943099 +++

Comment 1

[Dylan Oliver [:doliver]]

Attachment: notes.bn-BD.properties

This translation was provided by m@github (originally as https://github.com/mozilla-b2g/notes/pull/41/).

Will upload these values to the Transifex project and then re-export into the Notes repo.

Comment 2

[Dylan Oliver [:doliver]]

Oops, credit goes to maktrix @ github.

Comment 3

[Mahay Alam Khan]

(In reply to Dylan Oliver [:doliver] from comment #1)
> Created attachment 8461139 [details]
> notes.bn-BD.properties
> 
> This translation was provided by m@github (originally as
> https://github.com/mozilla-b2g/notes/pull/41/).
> 
> Will upload these values to the Transifex project and then re-export into
> the Notes repo.

Sorry my mistake, in a hurry I edited the en-US file instead of created a new one for bn-BD. Thank you for being careful and not merging right away. 

I have transifex account https://www.transifex.com/accounts/profile/maktrix/ let me know the URL for me to upgrade translations, just in case if there is any spelling mistake or I could do better if truncated.

Comment 4

[Dylan Oliver [:doliver]]

The Transifex project is here: https://www.transifex.com/projects/p/notes/

I have uploaded your strings from the pull request so you might want to check them for correctness just to ensure that I got everything right in the conversion process. There are also two more strings that we need for the manifest file and then I think we'll be all set.

Comment 5

[Dylan Oliver [:doliver]]

Attachment: PR - add bn-BD l10n

Mahay, here's the new pull request to add the bn-BD file. You'll probably want to take a look at the "description" field in the manifest.webapp file and provide an alternate translation there. You can leave it in a comment here or on the pull request and I'll get it to the right places. 

Otherwise you should just need to review that the strings from your original pull request made it up to Transifex and back into this new file OK.

Comment 6

[Dylan Oliver [:doliver]]

ni? to l10n -- is there someone available who can take this review? (See comment #5 for notes.)

Comment 7

[Axel Hecht [:Pike]]

I'd point you to MAK, so you got the right person here.

MAK, did you do this kind of review process before? If not, here's how that works. First, you view the attachment, bugzilla will redirect you there when you visit https://bugzilla.mozilla.org/attachment.cgi?id=8462877. Check the changes in the "Files changed" tab, leave comments on the lines if needed.

Then go to the details, https://bugzilla.mozilla.org/attachment.cgi?id=8462877&action=edit, and set the 'review' flag to either + or -, depending on whether the change should land as is or not. It's also good to recap if you left comments in the pull request UI.

Comment 8

[Dylan Oliver [:doliver]]

MAK, can I assume r+ once I make the change you left on the pull request?

Comment 9

[Dylan Oliver [:doliver]]

Comment on attachment 8462877 [details] [review]
PR - add bn-BD l10n

Updated PR with new description, ready for another look.

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Master: https://github.com/mozilla-b2g/notes/commit/206046d3bddcdc2de33bbe7ee9e60d76c538ed87