Skip to content

Conversation

@boljsa
Copy link

@boljsa boljsa commented Apr 20, 2016

Added Croatian translation - part one

Franjo Boljkovac added 5 commits April 20, 2016 11:48
Added Croatian
Added Croatian
Added Croatian
Added Croatian index.html
Added Croatian
@boljsa boljsa closed this Apr 20, 2016
@boljsa boljsa reopened this Apr 21, 2016
@boljsa boljsa closed this Apr 21, 2016
@boljsa
Copy link
Author

boljsa commented Apr 26, 2016

Should I close the pull request or someone else?
Who will review and merge this changes?
Thanks for the reply.

@ulyssepence
Copy link
Contributor

Hi @boljsa,

Thank you and good effort, but you've only translated the front page (index.html). I think it is premature to accept this pull request without at least one lesson (basics.textile for example).

Also, Croatian fluency is somewhat rare at Twitter internally so it's possible that we won't be able to find someone to review your changes.

@ulyssepence
Copy link
Contributor

Oh I didn't see your other pull request: #168 . I will continue discussion there

@boljsa
Copy link
Author

boljsa commented Apr 27, 2016

Hi Ben,

Thank you for your response.
I will continue to translate files and add them to the pull request #168 if you agree.

Best regards,
Frank (boljsa)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants