Skip to content

feature(translation): add Spanish translation #442

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Oct 19, 2017

Conversation

khvilaboa
Copy link
Contributor

It is a first translation. I probably will improve it in the next days but any review is welcomed.

Regards!

@lex111
Copy link
Member

lex111 commented Oct 7, 2017

@khvilaboa you watched PR #282?

@lex111 lex111 requested a review from alejandronanez October 7, 2017 00:26
// Hash: 01ae4880b9b4296907d680e9b909713c36c42491
// Link: https://github.com/gitpoint/git-point/pull/260

export const en = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be es

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed

@khvilaboa
Copy link
Contributor Author

@lex111 Yes, I had taken it into account.

@Rishabh04-02
Copy link

@khvilaboa This will pass all tests if you will change double quotes " to single quote ' in the following lines - L28 , L86 , L105 , L108 , L110 , L115 , L138

Suggestion based on the TravisCI Errors Log

@andrewda andrewda changed the title Spanish translation feature(translation): add Spanish translation Oct 7, 2017
@alejandronanez
Copy link
Member

There's a PR already open for this feature. I'm merging it so we don't get any problems later. 👍

},
pullMerge: {
title: 'Fusionar Pull Request',
createMergeCommit: 'Create a merge commit',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You missed this one.

title: 'Fusionar Pull Request',
createMergeCommit: 'Create a merge commit',
squashAndMerge: 'Squash y Merge',
merge: 'fusionar',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should leave squash and merge and any git reserved word in English. What you guys think @lex111 @khvilaboa @andrewda @housseindjirdeh ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alejandronanez it all depends on how you decided to translate in your language these phrases..

For example, you can see how this is done in the book on Git, the Spanish version.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alejandronanez I commited a version with the git reserved words without translate (I have seen that other translations follow this pattern).

@khvilaboa
Copy link
Contributor Author

@andrewda @alejandronanez What do you think of the current version?

"This means that in no way, shape or form do we ever view, use or share a user's GitHub data. If private data ever becomes visible at any point we will not record or view it. If it happens to be accidentally recorded, we will delete it immediately using secure erase methods. Again, we've set up authentication specifically so that this never happens.",
analyticsInfoTitle: 'ANALYTICS INFORMATION',
'Esto significa que de ninguna manera miramos, usamos o compartimos datos de GitHub de los usuarios. Si la información privada se volviera visible en algún momento, no la registraremos ni leeremos. Si se guardara accidentalmente, la borraremos inmediatamente usando métodos seguros de borrado. Sin embargo, hemos configurado la autenticación para que esto nunca suceda.',
analyticsInfoTitle: 'INFORMACIÓN ANALÍTICA',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think Información de Analytics would be better.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@khvilaboa can you please update this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alejandronanez Updated

Copy link
Member

@alejandronanez alejandronanez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove english.

analyticsInfo1:
'We currently use Google Analytics and iTunes App Analytics to help us measure traffic and usage trends for the GitPoint. These tools collect information sent by your device including device and platform version, region and referrer. This information cannot reasonably be used to identify any particular individual user and no personal information is extracted.',
'Actualmente usamos Google Analytics y iTunes App Analytics para ayudarnos a medir el tráfico y las tendencias de uso de GitPoint. Estas herramientas recogen información mandada por tu dispositivo, invluyendo la versión del dispositivo y la plataforma, zona geográfica y These tools collect information sent by your device including device and platform version, region and referente. Esta información no puede ser usada para identificar ningún usuario individual y no se extrae ninguna información personal.',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this.
These tools collect information sent by your device including device and platform version, region and referente.

@khvilaboa
Copy link
Contributor Author

@alejandronanez English text removed

stars: 'Estrellas',
orgs: 'ORGANIZACIONES',
noOrgsMessage: 'No hay organizaciones',
info: 'INFORMACIÖN',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

INFORMACIÓN

openButton: 'Abiertos',
closedButton: 'Cerrados',
searchingMessage: 'Buscando {{query}}',
noOpenPulls: 'No se encontraron pull requests abiertos!',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Start with ¡.

closedButton: 'Cerrados',
searchingMessage: 'Buscando {{query}}',
noOpenPulls: 'No se encontraron pull requests abiertos!',
noClosedPulls: 'No se encontraron pull requests cerrados!',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Start with ¡.

closedButton: 'Cerradas',
searchingMessage: 'Buscando {{query}}',
noOpenIssues: 'No se encontraron issues abiertas!',
noClosedIssues: 'No se encontraron issues cerradas!',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Start with ¡.

openButton: 'Abiertas',
closedButton: 'Cerradas',
searchingMessage: 'Buscando {{query}}',
noOpenIssues: 'No se encontraron issues abiertas!',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Start with ¡.

codePushCheck: 'Comprobar actualizaciones',
codePushChecking: 'Comprobando actualizaciones...',
codePushUpdated: 'Aplicación actualizada',
codePushAvailable: 'Actualizaciones disponibles!',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Start with ¡.

commitCommentEvent: 'commented on commit',
createEvent: 'creado {{object}}',
deleteEvent: 'borrado {{object}}',
'Bienvenido! Este es tu hilo de noticias - te ayudará a mantenerte informado sobre la actividad de los respositorios y los usuarios que sigues.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Start with ¡.

@khvilaboa
Copy link
Contributor Author

@ocarreterom Thanks for the review! Translation updated.

@khvilaboa
Copy link
Contributor Author

@andrewda What do you think of the current translation?

Copy link
Contributor

@ocarreterom ocarreterom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spanish labels and titles are not in Camel Case:
img_866dba02b245-1
img_635d47536e52-1

openSource1:
'GitPoint is open source and the history of contributions to the platform will always be visible to the public.',
'GitPoint es código abierto y el histórico de contribuciones a la plataforma será siempre visible al público.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GitPoint es de código abierto [...]

Copy link
Member

@chinesedfan chinesedfan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@khvilaboa Thanks for your work! I find some fields are missing compared with en.js. Can you add them?

numFilesChanged: '{{numFilesChanged}} archivos',
new: 'NUEVOS',
deleted: 'ELIMINADOS',
fileRenamed: 'Archivo renombrado sin cambios',
},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@khvilaboa Please add these fields.

    readMe: {
      readMeActions: 'README Actions',
      noReadMeFound: 'No README.md found',
    },

membersTitle: 'MEMBERS',
descriptionTitle: 'DESCRIPTION',
membersTitle: 'MIEMBROS',
descriptionTitle: 'DESCRIPCIÓN',
},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

organizationActions: 'Organization Actions',

merged: 'Merged',
},
screenTitles: {
issue: 'Issue',
pullRequest: 'Pull Request',
},
openIssueSubTitle: '#{{number}} abiertas hace {{time}} por {{user}}',
closedIssueSubTitle: '#{{number}} por {{user}} fue cerrada hace {{time}}',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

issueActions: 'Issue Actions',

@@ -318,5 +331,8 @@ export const erase = {
y: '%dy',
yy: '%dy',
},
abbreviations: {
thousand: 'k',
},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

openInBrowser: 'Open in Browser',

@khvilaboa
Copy link
Contributor Author

@chinesedfan Fields added. Thanks!

# Conflicts:
#	src/locale/languages/es.js
@chinesedfan
Copy link
Member

@khvilaboa Good. And can you also fix those ESLint errors? Some tabs is mixed into source codes.

/home/travis/build/gitpoint/git-point/src/locale/languages/es.js
  242:2  error  Unexpected tab character  no-tabs
  286:2  error  Unexpected tab character  no-tabs
  286:2  error  Mixed spaces and tabs     no-mixed-spaces-and-tabs
  287:2  error  Unexpected tab character  no-tabs
  343:2  error  Unexpected tab character  no-tabs

Copy link
Member

@housseindjirdeh housseindjirdeh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a million for this @khvilaboa. Whenever you get the chance to fix those minor linting issues and resolve the few conflicts, we'll merge this beauty in :)

@khvilaboa
Copy link
Contributor Author

@housseindjirdeh Conflicts resolved. Regards!

Copy link
Contributor

@ocarreterom ocarreterom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found some words than should be downcase and a field without translation.

writeATitle: 'Write a title for your issue here',
issueComment: 'Issue Comment',
writeAComment: 'Write a comment for your issue here',
title: 'Nueva Issue',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nueva issue

createMergeCommit: 'Create a merge commit',
squashAndMerge: 'Squash and merge',
createMergeCommit: 'Crear un commit merge',
squashAndMerge: 'Squash y Merge',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Squash y merge

writeATitle: 'Escribe aquí un título para tu commit',
commitMessage: 'Mensaje del commit',
writeAMessage: 'Escribe aquí un mensaje para tu commit',
mergeType: 'Tipo de Merge',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tipo de merge

commitMessage: 'Mensaje del commit',
writeAMessage: 'Escribe aquí un mensaje para tu commit',
mergeType: 'Tipo de Merge',
changeMergeType: 'Cambiar Tipo de Merge',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cambiar tipo de merge

following: 'Following',
cancel: 'Cancel',
yes: 'Yes',
website: 'Sitio Web',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sitio web

},
organizationActions: 'Organization Actions',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs translate.

newIssueButton: 'New Issue',
noIssuesMessage: 'No hay issues',
noOpenIssuesMessage: 'No hay issues abiertas',
viewAllButton: 'Ver Todo',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ver todo

noContributorsMessage: 'No se encontraron contribuidores',
sourceTitle: 'CÓDIGO',
readMe: 'LEEME',
viewSource: 'Ver Código',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ver código

repoActions: 'Acciones del repositorio',
forkAction: 'Hacer fork',
starAction: 'Marcar Favorito',
unstarAction: 'Desmarcar Favorito',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Desmarcar favorito

shareRepositoryMessage: 'Revisa {{repoName}} en GitHub. {{repoUrl}}',
repoActions: 'Acciones del repositorio',
forkAction: 'Hacer fork',
starAction: 'Marcar Favorito',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Marcar favorito

@khvilaboa
Copy link
Contributor Author

@ocarreterom Done. Thanks!

@andrewda andrewda added this to the Release 1.4.0 milestone Oct 17, 2017
@housseindjirdeh
Copy link
Member

housseindjirdeh commented Oct 19, 2017

THANK YOU @khvilaboa @ocarreterom

This is going to be amazing to have 🇪🇸 🎉

@housseindjirdeh housseindjirdeh merged commit 82a5f47 into gitpoint:master Oct 19, 2017
dyesseyumba pushed a commit to dyesseyumba/git-point that referenced this pull request Oct 20, 2017
* Spanish file (first translation)

* Languague related files

* Fix

* Replaced double quotes

* Improved spanish translation

* Minor improvements (spanish translation)

* Some improvements (spanish translation)

The GIT reserved words are kept without translate.

* Removed english phrase

* Updated analytics title

* Updated spanish translation

* Added missing fields (spanish translation)

* Replaced tabs (spanish translation)

* Changed some words to downcase (spanish translation)
chinesedfan pushed a commit that referenced this pull request Jan 6, 2018
* refactor(fonts): Remove useless fonts in android (#485)

* refactor(fonts): Remove MaterialIcons from used fonts in android (#485)

BREAKING CHANGE: Update link script in Package.json

* Revert "refactor(fonts): Remove MaterialIcons from used fonts in android (#485)"

This reverts commit 282f475.

* fix: Update stateRandom and reset cookies after a successful login (#494)

* feat(markdown): Add support for quoted emails (#501)

* feat(markdown): Add support for quoted emails

* fix: use paddingHorizontal instead of Left and Right

* refactor: Drop rn-app-intro in favor of react-native-swiper (#493)

* refactor: Drop rn-app-intro in favor of react-native-swiper

* fix: Don't embed swiper in a View, so that it works on Android

* chore: Hide the commitlint folder (#488)

* feature(translation): add Spanish translation (#442)

* Spanish file (first translation)

* Languague related files

* Fix

* Replaced double quotes

* Improved spanish translation

* Minor improvements (spanish translation)

* Some improvements (spanish translation)

The GIT reserved words are kept without translate.

* Removed english phrase

* Updated analytics title

* Updated spanish translation

* Added missing fields (spanish translation)

* Replaced tabs (spanish translation)

* Changed some words to downcase (spanish translation)

* feat: Issue Events (#438)

* feat(issue_events): Show events on issues

* style(issue_events): Added styles to "added label" event

* style(issue_events): Add icon & improve styling of added labels

* style(issue_events): Improve <ReviewRequested /> styles

* feat(issue_events): Remove mentioned/subscribed events from UI

* feat(issue_events): Define <Closed /> events

* refactor(issue_events): Extract <EventIcon /> and <Date />

* feat(issue_events): Add `unlabled` prop to <Labeled />

* feat(issue_events): Define <Merged /> event

* feat(issue_events): Filter out `closed` events preceded by `merged`

* feat(issue_events): Define <HeadRef /> events

* feat(issue_events): Define <Assigned /> events

* feat(issue_events): Define <Reopened /> & <Renamed /> events

* refactor(issue_events): Render <Text /> from <ActorLink />

* style(issue_events): Trim issue names to ensure spacing

* feat(issue_events): Define <Locked />  event

* feat(issue_events): Define <Milestoned />  event

* refactor(issue_events): Clean up authUser from LabeledComponent

* feat(issue_events): Define <MarkedAsDuplicate /> event

* refactor(issue_events): Define generic <Event /> component

* docs(readme): Add @brandly as a contributor

* feat(issue_events): Define <LabelGroup /> for list of label changes

* refactor(issue_events): Use spread operator for textChildren

* style(issue_events): Add blank line after external imports

* feat(issue_events): <InlineLabel /> has rounded corners

* refactor(issue_events): Move <InlineLabel /> into own file

* feat(issue_events): Press username in events to view profile

* refactor(events): Inline most <Event />s into <IssueEventListItem />

* refactor(events): Eliminate <Date /> since its only used once

* refactor(events): Extract formatEventsToRender into event-helpers

* fix(ux): Add back button for AuthProfileScreen (#507)

Adds back button when AuthProfileScreen is not the root of a StackNavigator.

Ps. AuthProfileScreen is StackNavigator root when the routeName is MyProfile.

* style(issueeventlistitem, commentlistitem): Slightly shrink issue event badges + change user click o (#516)

* fix: Remove undefined var & fix typo (#517)

* chore: fix `yarn run link` (#513)

* chore(fonts): Not link all fonts from react-native-vector-icons

* fix(fonts): Add the missing Menlo

* fix(cli): Fix for RN 0.48

* chore(cli): Run `yarn run link` again

* chore(*): convert notification icon styles to styled-component (#510)

Convert notification-icon.component.js to use styled-components as part of #503

* chore(*): convert label-list-item component styles to styled component (#509)

Converted the label-list-item component styles to styled-components

* chore(*): style view-container.component.js (#508)

Converted view-container.component.js to use styled components

BREAKING CHANGE: none

none

* refactor(auth): get user data after login (#502)

* test: begin implementing basic component tests (#407)

* test: begin implementing tests

* test: add more tests

* test: add Buton and Badge tests

* test: add StateBadge test

* test: only import used enzyme wrappers

* chore(deps): update table component to 1.1.0

* test: Add tests for ToggleView

* Update ToggleView.js

* refactor(fonts): Remove useless fonts in android (#485)

* refactor(fonts): Remove MaterialIcons from used fonts in android (#485)

BREAKING CHANGE: Update link script in Package.json

* Revert "refactor(fonts): Remove MaterialIcons from used fonts in android (#485)"

This reverts commit 282f475.

* test: Add tests for CommentInput (#518)

* refactor: Beautify the code of CommentInput unit test.

* refactor: use jest mocks instead of sinon spies.

* refactor: Apply @chinesedfan recommendations

* refactor: Improve test descriptions.

* test: Add two cases of test and integrate styled-components in tests.

Add two more cases for userHasPushPermission and issueLocked. Update test to use migrated components
to styled-components.

* refactor: Remove console.log statement.

* test: Improve test descriptions and remove useless console.log

* test: Fix conflicts between descriptions and implementations.

* refactor: Runned prettier on CommentInput.js
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.

8 participants