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

Sticky flag "y", searching at position#424

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 11 commits intojavascript-tutorial:masterfromAbiF73:master
Dec 17, 2020

Conversation

AbiF73
Copy link
Contributor

Hola! Tuve algunas dudas con algunas palabras, lo siento. Espero sus correcciones.

@CLAassistant
Copy link

CLAassistant commentedOct 15, 2020•
edited
Loading

CLA assistant check
All committers have signed the CLA.

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.

bienvenido
y bienvenido si continuas

Sabes de git ? veo que modificaste tu master
No nos afecta en el pr, pero lo mejor para trabajar en tu fork es
dejar el master intocable (solo actualizar desde nuestro fork)
generar en un branch y modificar alli
una vez hecho el pr y añadido al nuestro repo, puedes borrar el branch
y actualizar tu master desde nuestro remoto


We can get all matches in the loop:
Podemos conseguir todas las coincidencias en el loop:

```js run
Copy link
Member

Choose a reason for hiding this comment

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

¿Podrias traducir comentarios lineas 63 64 ?
github no me deja sugerir ahi (dentro de un bloque sin traducir)

Quizas tambien la 62 pero con cuidado:

alert(Se encontró ${result[0]} en la posición ${result.index} );

@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 titleUpdate article.mdSticky flag "y", searching at positionOct 15, 2020
@joaquinelio
Copy link
Member

ah, para llevar el control (y Don Bot lo entienda)
cambié el nombre al PR

Sticky flag "y", searching at position

@joaquinelio
Copy link
Member

AAAHHHHHHH!!!!

Ilya marcó la version en ingles como WIP
el crazy va a rescribirlo todo!!!!

@vplentinax
Copy link
Contributor

Después que se acepten las sugerencias de Joaquinelio, paso yo a revisar.

@vplentinaxvplentinax self-requested a reviewOctober 25, 2020 16:08
@joaquinelio
Copy link
Member

Después que se acepten las sugerencias de Joaquinelio, paso yo a revisar.

@vplentinax
El pr es de hace 10 dias, las suggest son de hace 10 dias...

Encima Ilya lo rescribió dos dias después.
Aun no se si hacer sync (ingles-ingles, seria directo) y que este quede con conflictos multiples
o esperar al merge de esto e descartar la nueva version para verla tranquilo

@vplentinax
Copy link
Contributor

@joaquinelio Hazlo como te parezca más cómodo a ti. No tengo inconveniente con ninguna de las dos formas.

@joaquinelio
Copy link
Member

@vplentinax
Este es un art breve
El ingles fue reescrito o rescrito, (en repo esp deje version vieja para que esto no diera conflictos)
podriamos descartar esto y hacerlo de cero

por otro lado,
corrigiendo ortografia le daria MERGE,
es un art muy facil de entender
e Ilya hizo mucho cambio no dire superfluo pero...

vplentinaxand others added9 commitsDecember 17, 2020 13:14
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
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 commit5944323 intojavascript-tutorial:masterDec 17, 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 requested changes

@vplentinaxvplentinaxvplentinax approved these changes

Assignees
No one assigned
Labels
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

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