Websocket by MHMighani · Pull Request #316 · javascript-tutorial/fa.javascript.info
-
Notifications
You must be signed in to change notification settings - Fork 105
Merged
Conversation
Translation for the websocket section
All committers have signed the CLA.
Comment on lines +263 to 266
| ## وضعیت اتصال | ||
|
|
||
| ## Connection state | ||
| برای اطلاع از وضعیت اتصال پراپرتی `socket.readyState` با مقادیر زیر وجود دارد: | ||
|
|
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check line numbers. the translation must be on the same line.
Comment on lines +281 to +282
| <input type="text" name="message" /> | ||
| <input type="submit" value="Send" /> |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please disable formatting for translations if you're using any. don't change code styles.
Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.