Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Fetch#435

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
joaquinelio merged 19 commits intojavascript-tutorial:masterfromcarlosabud:master
Oct 30, 2020
Merged

Fetch#435

joaquinelio merged 19 commits intojavascript-tutorial:masterfromcarlosabud:master
Oct 30, 2020

Conversation

carlosabud
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commentedOct 25, 2020•
edited
Loading

CLA assistant check
All committers have signed the CLA.

@joaquinelio
Copy link
Member

Hola
Todavia no lo revise pero te aviso que tiene ejercicios

uno ejercicio solo. en tu branch
https://github.com/carlosabud/es.javascript.info/tree/master/5-network/01-fetch/01-fetch-users
task.md y solution.md

Copy link
Member

@joaquineliojoaquinelio left a comment•
edited
Loading

Choose a reason for hiding this comment

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

  • Varios acentos, si no usas las suggest acordate de ponerlos
  • Recordar que faltan task y solution
  • Me permití corregir los nros de linea
    necesitamos que coincidan con el inglés para poder hacer bien los pull
  • Cambie nombre para que Don Bot lo reconozca, agregué el pr que Don Bot no reconoció al issue17

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@joaquineliojoaquinelio changed the titleTraducción articulo FetchFetchOct 25, 2020
@carlosabud
Copy link
ContributorAuthor

Buenas Joaquin, ahí realicé un par de actualizaciones tomando tus sugerencias.

Cualquier cosa avisame

@joaquinelio
Copy link
Member

Uh
Debí advertirte, no traducimos código, puede salir mal.
Y no lo podemos probar acá.
(Pero los devs somos perfectos y no cometemos errores, para qué probar)

Se traducen solamente // comentarios /* */
y usualmente mensajes entrecomillados (con cuidado los backticks) pero no los string que participan de codigo

salvo eso, el article creo que ya está, task y solution estan bien...
¿podrias poner codigo sin traducir?

carlosabud reacted with laugh emoji

@carlosabud
Copy link
ContributorAuthor

carlosabud commentedOct 26, 2020•
edited
Loading

@joaquinelio Creo que ahí estamos. Avisame!

Copy link
Member

@joaquineliojoaquinelio left a comment

Choose a reason for hiding this comment

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

Qudó un js
5-network/01-fetch/01-fetch-users/_js.view/solution.js
lo más seguro es poner el original entero,

al otro js le copie la linea vieja,
5-network/01-fetch/01-fetch-users/_js.view/source.js
por lo menos el comentario quedo traducido... =)

Estaba tentado a dejarlo asi, probarlo localmente
el problema es que a veces hacen cambios de codigo y el pull es directo
es posible por ej que agreguen una linea usando la variable ingles y nadie se de cuenta de que el codigo se malogro

Mas par de cositas,

carlosabud reacted with thumbs up emoji
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

carlosabudand others added4 commitsOctober 26, 2020 22:55
@carlosabud
Copy link
ContributorAuthor

Listo. Perdón por las idas y vueltas.

/done

Copy link
Member

@joaquineliojoaquinelio left a comment

Choose a reason for hiding this comment

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

👍

@carlosabud
Copy link
ContributorAuthor

@joaquinelio por las dudas, no me deja completar el PR. Supongo que hará falta que otra persona con permisos revise el PR y lo complete. Saludos!

@joaquinelio
Copy link
Member

Sí, solo maintainers pueden continuarlo.

  • El branch y el PR son tuyos, podes hacer y deshacer.
  • merge dentro del repo solo la puede hacer alguien con permisos,
  • la aprobación es cuestion de procedimiento, lo esperable es que dos personas aprueben y el bot de Ilya responde cambiando los cartelitos.
  • Vos podrias hacer review y aprobar el PR de otra persona, y aunque el BOT te ignore, el maintainter no
    (especialmente por la escasez de reviewers)

Ja, deberia poner esto en el curso de git filosofico
para complementar cualquier curso de comandos git

carlosabud reacted with laugh emoji

Copy link
Contributor

@vplentinaxvplentinax left a comment

Choose a reason for hiding this comment

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

Ok

@joaquineliojoaquinelio merged commita766c14 intojavascript-tutorial:masterOct 30, 2020
@javascript-translate-bot

Thank you 💖 I updated the Progress Issue#17 🎉 🎉 🎉

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

@joaquineliojoaquineliojoaquinelio approved these changes

@vplentinaxvplentinaxvplentinax approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

5 participants
@carlosabud@CLAassistant@joaquinelio@javascript-translate-bot@vplentinax
[8]ページ先頭
©2009-2025 Movatter.jp