GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — Long polling by homero304 · Pull Request #354 · javascript-tutorial/es.javascript.info

Conversation

homero304 commented Aug 22, 2020

Copy link

Copy Markdown

Contributor

Hi. Please review my PR.
question. How can i see the svg image in the article ?
I have the following error: /long-polling.svg: Could not execute GraphicsMagick/ImageMagick: identify "-ping" "-format" "{"width":%w,"height":%h}" "/long-polling.svg
I tried to install ImageMagick7 -- https://linuxconfig.org/how-to-install-imagemagick-7-on-ubuntu-18-04-linux
but ran into trouble when i tried to run the server again. It was complaining that it could not create a symbolic link for ImageMagick7++...la

Thanks
Regards,
Homero Marin

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions


          Long polling

a77934d

javascript-translate-bot added the review needed label

Aug 22, 2020


          Merge branch 'master' into Long-polling

810391b

iliakan mentioned this pull request

Aug 22, 2020

Spanish Translation Progress #17

Open

joaquinelio self-requested a review

August 22, 2020 23:03

joaquinelio commented Aug 22, 2020

Copy link

Copy Markdown

Member

Sobre img sobre server, ni idea. Igual no hace falta, Eze los tradujo todos, si no modificas el link funcionará.
Acá mismo lo podes probar "file changes" y en el menu "...", "view file"

¡Eh!

@iliakan iliakan mentioned this pull request 12 minutes ago Spanish Translation Progress #17

¡Fui yo quien lo agregó hace 12 minutos! Este bot más boleado cada día.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

joaquinelio requested changes

Aug 23, 2020

View reviewed changes

joaquinelio left a comment

Copy link

Copy Markdown

Member

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.

Algunas cosas...

que sugerí tambien cambiar en inglés
jeje Apuesto que lo rechaza pero que lo rescribe uno que sí sabe inglés.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

Comment thread

5-network/10-long-polling/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

5-network/10-long-polling/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

5-network/10-long-polling/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

5-network/10-long-polling/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

5-network/10-long-polling/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

5-network/10-long-polling/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

5-network/10-long-polling/longpoll.view/browser.js Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

5-network/10-long-polling/longpoll.view/server.js Resolved

Uh oh!

There was an error while loading. Please reload this page.

javascript-translate-bot added changes requested and removed review needed labels

Aug 23, 2020

javascript-translate-bot commented Aug 23, 2020

Copy link

Copy Markdown

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

omero329 and others added 9 commits

August 22, 2020 20:48


          Update 5-network/10-long-polling/article.md

a00d708

acentuacion de conjugacion estar en subjuntivo

Co-authored-by: joaquinelio <[email protected]>


          Update 5-network/10-long-polling/article.md

30108fa

cuesstión de percepcion...

Co-authored-by: joaquinelio <[email protected]>


          Update 5-network/10-long-polling/article.md

dae638f

conexiones, procesos y memoria consumida

Co-authored-by: joaquinelio <[email protected]>


          Update 5-network/10-long-polling/article.md

03a0d17

lenguajes, problema de implementacion no del lenguaje en si. algunos lenguajes son mas amigables que otros

Co-authored-by: joaquinelio <[email protected]>


          Update 5-network/10-long-polling/longpoll.view/browser.js

1372c70

pasado de expirar

Co-authored-by: joaquinelio <[email protected]>


          Update 5-network/10-long-polling/longpoll.view/server.js

8355a92

a -> para

Co-authored-by: joaquinelio <[email protected]>


          Update 5-network/10-long-polling/article.md

periodico

Co-authored-by: joaquinelio <[email protected]>


          error 502: tiempo de espera agotado

2f4d805


          cerró la conexión

882a263

homero304 commented Aug 23, 2020

Copy link

Copy Markdown

Contributor Author

/done

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

javascript-translate-bot added review needed and removed changes requested labels

Aug 23, 2020

joaquinelio approved these changes

Aug 23, 2020

View reviewed changes

joaquinelio left a comment

Copy link

Copy Markdown

Member

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.

👍

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

javascript-translate-bot added needs +1 and removed review needed labels

Aug 23, 2020

vplentinax approved these changes

Aug 27, 2020

View reviewed changes

vplentinax left a comment

Copy link

Copy Markdown

Contributor

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.

Ok!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

vplentinax added ready to merge and removed needs +1 labels

Aug 27, 2020

vplentinax merged commit fa5cee7 into javascript-tutorial:master

Aug 27, 2020

javascript-translate-bot commented Aug 27, 2020

Copy link

Copy Markdown

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Reviewers

joaquinelio

joaquinelio approved these changes

vplentinax

vplentinax approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

ready to merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.