-
Notifications
You must be signed in to change notification settings - Fork 783
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
Conversation
@khvilaboa you watched PR #282? |
src/locale/languages/es.js
Outdated
// Hash: 01ae4880b9b4296907d680e9b909713c36c42491 | ||
// Link: https://github.com/gitpoint/git-point/pull/260 | ||
|
||
export const en = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be es
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed
@lex111 Yes, I had taken it into account. |
@khvilaboa This will pass all tests if you will change Suggestion based on the TravisCI Errors Log |
There's a PR already open for this feature. I'm merging it so we don't get any problems later. 👍 |
src/locale/languages/es.js
Outdated
}, | ||
pullMerge: { | ||
title: 'Fusionar Pull Request', | ||
createMergeCommit: 'Create a merge commit', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You missed this one.
src/locale/languages/es.js
Outdated
title: 'Fusionar Pull Request', | ||
createMergeCommit: 'Create a merge commit', | ||
squashAndMerge: 'Squash y Merge', | ||
merge: 'fusionar', |
There was a problem hiding this comment.
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 ?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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).
The GIT reserved words are kept without translate.
@andrewda @alejandronanez What do you think of the current version? |
src/locale/languages/es.js
Outdated
"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', |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alejandronanez Updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove english.
src/locale/languages/es.js
Outdated
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.', |
There was a problem hiding this comment.
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.
@alejandronanez English text removed |
src/locale/languages/es.js
Outdated
stars: 'Estrellas', | ||
orgs: 'ORGANIZACIONES', | ||
noOrgsMessage: 'No hay organizaciones', | ||
info: 'INFORMACIÖN', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
INFORMACIÓN
src/locale/languages/es.js
Outdated
openButton: 'Abiertos', | ||
closedButton: 'Cerrados', | ||
searchingMessage: 'Buscando {{query}}', | ||
noOpenPulls: 'No se encontraron pull requests abiertos!', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Start with ¡
.
src/locale/languages/es.js
Outdated
closedButton: 'Cerrados', | ||
searchingMessage: 'Buscando {{query}}', | ||
noOpenPulls: 'No se encontraron pull requests abiertos!', | ||
noClosedPulls: 'No se encontraron pull requests cerrados!', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Start with ¡
.
src/locale/languages/es.js
Outdated
closedButton: 'Cerradas', | ||
searchingMessage: 'Buscando {{query}}', | ||
noOpenIssues: 'No se encontraron issues abiertas!', | ||
noClosedIssues: 'No se encontraron issues cerradas!', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Start with ¡
.
src/locale/languages/es.js
Outdated
openButton: 'Abiertas', | ||
closedButton: 'Cerradas', | ||
searchingMessage: 'Buscando {{query}}', | ||
noOpenIssues: 'No se encontraron issues abiertas!', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Start with ¡
.
src/locale/languages/es.js
Outdated
codePushCheck: 'Comprobar actualizaciones', | ||
codePushChecking: 'Comprobando actualizaciones...', | ||
codePushUpdated: 'Aplicación actualizada', | ||
codePushAvailable: 'Actualizaciones disponibles!', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Start with ¡
.
src/locale/languages/es.js
Outdated
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.', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Start with ¡
.
@ocarreterom Thanks for the review! Translation updated. |
@andrewda What do you think of the current translation? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
src/locale/languages/es.js
Outdated
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.', |
There was a problem hiding this comment.
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 [...]
There was a problem hiding this 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', | ||
}, |
There was a problem hiding this comment.
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', | ||
}, |
There was a problem hiding this comment.
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}}', |
There was a problem hiding this comment.
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', | |||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
openInBrowser: 'Open in Browser',
@chinesedfan Fields added. Thanks! |
# Conflicts: # src/locale/languages/es.js
@khvilaboa Good. And can you also fix those ESLint errors? Some tabs is mixed into source codes.
|
There was a problem hiding this 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 :)
@housseindjirdeh Conflicts resolved. Regards! |
There was a problem hiding this 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.
src/locale/languages/es.js
Outdated
writeATitle: 'Write a title for your issue here', | ||
issueComment: 'Issue Comment', | ||
writeAComment: 'Write a comment for your issue here', | ||
title: 'Nueva Issue', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nueva issue
src/locale/languages/es.js
Outdated
createMergeCommit: 'Create a merge commit', | ||
squashAndMerge: 'Squash and merge', | ||
createMergeCommit: 'Crear un commit merge', | ||
squashAndMerge: 'Squash y Merge', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Squash y merge
src/locale/languages/es.js
Outdated
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', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tipo de merge
src/locale/languages/es.js
Outdated
commitMessage: 'Mensaje del commit', | ||
writeAMessage: 'Escribe aquí un mensaje para tu commit', | ||
mergeType: 'Tipo de Merge', | ||
changeMergeType: 'Cambiar Tipo de Merge', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cambiar tipo de merge
src/locale/languages/es.js
Outdated
following: 'Following', | ||
cancel: 'Cancel', | ||
yes: 'Yes', | ||
website: 'Sitio Web', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sitio web
src/locale/languages/es.js
Outdated
}, | ||
organizationActions: 'Organization Actions', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs translate.
src/locale/languages/es.js
Outdated
newIssueButton: 'New Issue', | ||
noIssuesMessage: 'No hay issues', | ||
noOpenIssuesMessage: 'No hay issues abiertas', | ||
viewAllButton: 'Ver Todo', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ver todo
src/locale/languages/es.js
Outdated
noContributorsMessage: 'No se encontraron contribuidores', | ||
sourceTitle: 'CÓDIGO', | ||
readMe: 'LEEME', | ||
viewSource: 'Ver Código', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ver código
src/locale/languages/es.js
Outdated
repoActions: 'Acciones del repositorio', | ||
forkAction: 'Hacer fork', | ||
starAction: 'Marcar Favorito', | ||
unstarAction: 'Desmarcar Favorito', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Desmarcar favorito
src/locale/languages/es.js
Outdated
shareRepositoryMessage: 'Revisa {{repoName}} en GitHub. {{repoUrl}}', | ||
repoActions: 'Acciones del repositorio', | ||
forkAction: 'Hacer fork', | ||
starAction: 'Marcar Favorito', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Marcar favorito
@ocarreterom Done. Thanks! |
THANK YOU @khvilaboa @ocarreterom This is going to be amazing to have 🇪🇸 🎉 |
* 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)
* 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
It is a first translation. I probably will improve it in the next days but any review is welcomed.
Regards!