Skip to content
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

IndexedDB #491

Merged
merged 16 commits into from Aug 7, 2021
Merged

IndexedDB #491

merged 16 commits into from Aug 7, 2021

Conversation

joaquinelio
Copy link
Member

No description provided.

@joaquinelio
Copy link
Member Author

Trabajé con indexeddb (Crudo sin idb) hace un par de años,
igual la traduccion fue ardua
seria bueno una revision de un vet

igual le tengo confianza
es preferible que este online

El articulo original es execlente,
y saberlo te permite enfrentar cualquiera de las bases mas gordas de server
menos el sql,
pero claves, indices, transacciones, cursores, TODO esta ahi, explicadisimo

@joaquinelio
Copy link
Member Author

ppffff por mi culpa ilya rescribio varias partes
7 conflictos
hecho.

@joaquinelio
Copy link
Member Author

8 conflictos nuevos, hecho.

@joaquinelio
Copy link
Member Author

2 conflictos resueltos
trabajo ocioso, deberia estar online

@joaquinelio
Copy link
Member Author

cobflicto resuelto
mine wins

Copy link
Contributor

@vrivas vrivas left a comment

Choose a reason for hiding this comment

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

He realizado solo tres modificaciones para ver si la forma en la que lo hago es correcta.

6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
@joaquinelio
Copy link
Member Author

joaquinelio commented Aug 2, 2021

se pueden hacer varias revisiones,

para que el bot me avise pones "finish review" pero con "change req"
luego hago (o descarto/desprecio) los cambios, y pongo /done
entonces el bot te avisa para que re-revises

edito
el bot fue muy util para administrar el curso y seguimiento de las traducciones
Don Bot es de este repo este solamente,

me registre para que github me notifiqie todos los eventos
tipicamente si usas el inbox de las notificaciones pendientes es ms q suficiente para trabajar

joaquinelio and others added 3 commits August 2, 2021 09:31
Co-authored-by: Víctor Manuel Rivas Santos <vrivas@users.noreply.github.com>
Co-authored-by: Víctor Manuel Rivas Santos <vrivas@users.noreply.github.com>
Co-authored-by: Víctor Manuel Rivas Santos <vrivas@users.noreply.github.com>
@joaquinelio
Copy link
Member Author

joaquinelio commented Aug 2, 2021

1 ¡Excelentes los cambios de redacción! Estaban algo pegoteados al ingles. No creo que mal, pero ahora es mucho más legible y es lo que importa
2 "ésto" no se acentua
3 me hiciste avergonzar con esa coma colgada, siempre las critico y no es de las que perdono.
4 no entendí poderosa/potente pero como me da igual lo acepté.
Una vez se burlaron mundialmente porque puse "saltear" en lugar de "saltar". Aunque RAE registra "saltear = omitir" como americanismo, América también se burló de mí. Abatido.

podes continuar, plis!

Copy link
Contributor

@vrivas vrivas left a comment

Choose a reason for hiding this comment

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

No es fácil :)

6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
Co-authored-by: Víctor Manuel Rivas Santos <vrivas@users.noreply.github.com>
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

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

@joaquinelio
Copy link
Member Author

esto ya atenta con la autoestima, crei qu mi fuerte era el buen uso del idioma
fue creo el articulo mas arduo de los que traduje

aplique rapidamente solamente las evidentes (asi oculto las que me avergonzaban ja)
alguna considero quedo con error,
otras tengo que meditarlas, no gramatica sino conceptos y tiempos secuencias
una esta perfecta pero tengo duda de origen, cometi pecado de reinterpretarlo y desvie el original
vuelvo mas tarde

aqui.
sigo con la intriga, siempre lo asumi como la minima localidad que permite el contexto,
si no , es una palabra que no tiene razon de existencia
si uno dice
"aqui y ahora
casi esta viendo el dedo apuntando al piso

@joaquinelio
Copy link
Member Author

hola @jespadas
sorpresa, 5 meses sin revisores y hoy cumplimos con los dos x PR
te presento a @vrivas probablemente conmejor manejo de idioma que yo

diviertanse vuelvo luego
quizas prefieran a que uno termine antes de seguir el otro

@vrivas
Copy link
Contributor

vrivas commented Aug 3, 2021

aqui.
sigo con la intriga, siempre lo asumi como la minima localidad que permite el contexto,
si no , es una palabra que no tiene razon de existencia
si uno dice
"aqui y ahora
casi esta viendo el dedo apuntando al piso

Creo que es porque quizá le damos a "aquí" una interpretación muy "física", de un lugar físico concreto, y no tanto de un documento.

No le des muchas más vueltas, si te cuadra "aquí", no hay por qué cambiarlo.

@vrivas
Copy link
Contributor

vrivas commented Aug 3, 2021

hola @jespadas

hola @jespadas

sorpresa, 5 meses sin revisores y hoy cumplimos con los dos x PR

Es lo que tienen las vacaciones en agosto :)

te presento a @vrivas probablemente conmejor manejo de idioma que yo

heso ez dixcutivle :D

diviertanse vuelvo luego
quizas prefieran a que uno termine antes de seguir el otro

La revisión de #495 también está pendiente. ¿Qué me dices @jespadas? ¿sigues tú con esta (#491) y tomo yo la #495? ¿O lo hacemos al revés?

@joaquinelio
Copy link
Member Author

joaquinelio commented Aug 4, 2021

la superposición no es problema en sí, el procedimiento prevé 2 revisores por traduccion

podemos apegarnos al espiritu del sistema "lo que me de la gana"

@jespadas, habia pedido ayuda con revs porque no esperaba tal abrumadora multitud de 2, si seguis con ganas de traducir, es lo que sugiero

@vrivas viniste dispuesto a revisar, pues podes seguir ayudando si es tu deseo

yo en cambio que no tengo ganas de hacer nada,, apenas vere si susy tiene algo que valga, no mas.
(*susy = super-sync, merge desde repo ingles)

por las dudas tamb sugiero
github tiene el centro de notificaciones (campanita)
el inbox ahi es excelente para estar al tanto de lo q uno tiene pendiente leer/hacer

Co-authored-by: Víctor Manuel Rivas Santos <vrivas@users.noreply.github.com>
@jespadas
Copy link
Contributor

jespadas commented Aug 4, 2021

hola @jespadas

hola @jespadas

sorpresa, 5 meses sin revisores y hoy cumplimos con los dos x PR

Es lo que tienen las vacaciones en agosto :)

te presento a @vrivas probablemente conmejor manejo de idioma que yo

heso ez dixcutivle :D

diviertanse vuelvo luego
quizas prefieran a que uno termine antes de seguir el otro

La revisión de #495 también está pendiente. ¿Qué me dices @jespadas? ¿sigues tú con esta (#491) y tomo yo la #495? ¿O lo hacemos al revés?

Tomo la #495 !

Copy link
Contributor

@vrivas vrivas left a comment

Choose a reason for hiding this comment

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

Creo que ya he terminado la revisión "gorda".
Seguro que si volvemos a leerla, saldrán muchas cosas más que revisar... pero no se trata de entrar en un bucle infinito.
@joaquinelio Acepta/rechaza lo que veas claro; discutimos lo que haya dudas; y publicamos cuanto antes.

¡Animo!

6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
6-data-storage/03-indexeddb/article.md Outdated Show resolved Hide resolved
joaquinelio and others added 2 commits August 7, 2021 16:47
Co-authored-by: Víctor Manuel Rivas Santos <vrivas@users.noreply.github.com>
@joaquinelio
Copy link
Member Author

@vrivas
¡Ya pero YA! No máas de un semestre más.
muy buena rev, edite un par de cositas.

cuando me quejo de laas comas,no es especiicamente las comas sino la contruccion.
fuiste eficaz lidiando con eso, no es solo gramatica lo q arreglaste

MERGE
sig suggest, pr a master
poodes mirar este pr para comparar o mirarlo online para una rev fresca... o para admirarlo nomas.

@joaquinelio joaquinelio merged commit ee3cc89 into javascript-tutorial:master Aug 7, 2021
@javascript-translate-bot

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

@vrivas
Copy link
Contributor

vrivas commented Aug 7, 2021

¡Excelente trabajo, @joaquinelio!
Ha sido un placer y he aprendido muchísimo.
Intentaré seguir revisando otros capítulos.

@joaquinelio joaquinelio deleted the idb branch August 8, 2021 03:02
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.

None yet

4 participants