Skip to content

WeakMap and WeakSet - #162

Merged
tarasyyyk merged 17 commits into
javascript-tutorial:masterfrom
MykolaSopiha:weakmap-weakset
Aug 3, 2021
Merged

WeakMap and WeakSet#162
tarasyyyk merged 17 commits into
javascript-tutorial:masterfrom
MykolaSopiha:weakmap-weakset

Conversation

@MykolaSopiha

@MykolaSopihaMykolaSopiha commented Jul 28, 2021

Copy link
Copy Markdown
Collaborator

Під час перекладу помітив, що інколи назви методів перелічуються в тексті з дужками, а іноді ні. Навіть в одному і тому ж реченні. Можливо це слід обговорити та прийти до єдиного стандарту. Поки що перекладав згідно з оригіналом:
image

Comment thread1-js/05-data-types/08-weakmap-weakset/01-recipients-read/solution.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/01-recipients-read/solution.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/01-recipients-read/task.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/01-recipients-read/task.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/02-recipients-when-read/task.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/02-recipients-when-read/task.md Outdated

@tarasyyyktarasyyyk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Дуже добре. Є декілька невеличких правок. Я вже додав пропозиції, тож залишилося лише їх закомітити. На рахунок назв методів. Про це не переймайтеся – пізніше в англійському варіанті їх заберуть і я оновлю переклад.

Comment thread1-js/05-data-types/08-weakmap-weakset/article.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/article.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/article.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/article.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/article.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/article.md Outdated
Comment thread1-js/05-data-types/08-weakmap-weakset/article.md Outdated
@javascript-translate-bot

Copy link
Copy Markdown
Contributor

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

MykolaSopihaand others added 9 commits July 29, 2021 22:59
Co-authored-by: Taras <oster.ok@gmail.com>
Co-authored-by: Taras <oster.ok@gmail.com>
Co-authored-by: Taras <oster.ok@gmail.com>
Co-authored-by: Taras <oster.ok@gmail.com>
Co-authored-by: Taras <oster.ok@gmail.com>
Co-authored-by: Taras <oster.ok@gmail.com>
Co-authored-by: Taras <oster.ok@gmail.com>
@MykolaSopiha

Copy link
Copy Markdown
CollaboratorAuthor

@tarasyyyk дякую за правки! Я їх закомітив + додав ще пару невеликих фіксів. Сподіваюсь тепер все ок)

@MykolaSopiha

Copy link
Copy Markdown
CollaboratorAuthor

/done

@tarasyyyk

Copy link
Copy Markdown
Collaborator

@MykolaSopiha чудово! Вибачте за затримку. Зараз в мене обмаль часу на перевірки.
Переклад підтверджую.

@tarasyyyk
tarasyyyk merged commit 89ee8e9 into javascript-tutorial:masterAug 3, 2021
@javascript-translate-bot

Copy link
Copy Markdown
Contributor

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

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@MykolaSopiha@javascript-translate-bot@tarasyyyk