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

6-data-storage/01-cookie #318

Conversation

Kichikahunov
Copy link
Contributor

No description provided.

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Jul 23, 2019
@javascript-translate-bot javascript-translate-bot requested a review from a team July 23, 2019 12:34
@iliakan
Copy link
Member

iliakan commented Jul 24, 2019

"куки, установленную сайтом site.com" - слово "куки" женского рода, выходит?

@Kichikahunov
Copy link
Contributor Author

@iliakan, выходит так...

@xcurveballx, что скажешь?

"Нет способа сделать куки доступной на другом домене 2-го уровня, так что other.com никогда не получит куки, установленную сайтом site.com"

@xcurveballx
Copy link
Member

@Kichikahunov если больше по тексту она нигде не употребляется в среднем или мужском роде - то ок. Тоже нормальный вариант. И как уже писалось в комментах под #74, для многих это естественно и привычно. Думаю, в итоге изобретать велосипед не стоит, раз есть уже сложившееся употребление, которое само приходит на ум при переводе.

@iliakan
Copy link
Member

iliakan commented Jul 24, 2019

@xcurveballx @Kichikahunov Цитата из википедии "Тем не менее, куки остается уязвимым для кросс-сайтовой трассировки (XST) и кросс-сайтовых поддельных запросов (XSRF)."

"уязвимым" - значит мужской или средний род?

есть еще авторитетные источники?

@iliakan
Copy link
Member

iliakan commented Jul 24, 2019

Там же "Эта куки может.."

Давайте сойдёмся на женском роде +

@Kichikahunov
Copy link
Contributor Author

согласен, женский род

@Kichikahunov
Copy link
Contributor Author

это может длиться вечно 🤣

@iliakan
Copy link
Member

iliakan commented Jul 25, 2019

вот тут в коментах ссылка на учебник: https://rus.stackexchange.com/questions/2536/%d0%9a%d0%b0%d0%ba%d0%be%d0%b9-%d1%80%d0%be%d0%b4-%d1%83-%d1%81%d0%bb%d0%be%d0%b2%d0%b0-%d1%81%d1%83%d1%88%d0%b8

Суши - среднего рода, по общему правилу - несклоняемые заимствованные существительные,если они неодушевленные, относятся к среднему роду.

@iliakan
Copy link
Member

iliakan commented Jul 25, 2019

вот источник https://studopedia.info/6-31699.html

Наименования неодушевленных предметов чаще всего относятся к существительным среднего рода. Отступления от этого общего правила и колебания в родовом оформлении наблюдаются прежде всего в тех случаях, когда несклоняемые слова тесно связаны с семантически близкими обобщающими словами. Иногда в этих случаях побеждает род обобщающего слова, хотя в целом на протяжении 19-20 в.в. действует тенденция к устранению колебаний в пользу вариантов среднего рода.

@iliakan
Copy link
Member

iliakan commented Jul 25, 2019

Значит:

  • это одно куки
  • два куки

@iliakan
Copy link
Member

iliakan commented Jul 25, 2019

@xcurveballx @Kichikahunov есть консенсус?

@iliakan
Copy link
Member

iliakan commented Jul 25, 2019

@Kichikahunov похоже, это последняя статья (за исключением почти готовой selection/range), которую осталось сделать для публикации учебника =)

@Kichikahunov
Copy link
Contributor Author

@iliakan 80% перевел, ещё приложение и итого

потом перечитаю, поправлю все куки и вперёд

@Kichikahunov Kichikahunov marked this pull request as ready for review July 26, 2019 11:17
@Kichikahunov
Copy link
Contributor Author

Kichikahunov commented Jul 26, 2019

@iliakan, @xcurveballx, ну как-то так 🤗

Copy link
Member

@iliakan iliakan left a comment

Choose a reason for hiding this comment

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

Прими плиз правки, и также глянь - этот перевод основан на текущей англ версии или нет?

А то я смотрю там обороты в англ, которые были давно изменены. Правки к предыдущему PR ты учитывал или заново делал?

6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels Jul 27, 2019
@javascript-translate-bot
Copy link
Contributor

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

@iliakan iliakan added the partially reviewed Last review only took a part of the article. Full re-review (not just changes since last) required label Jul 27, 2019
Copy link
Member

@xcurveballx xcurveballx left a comment

Choose a reason for hiding this comment

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

И вроде там в папке есть еще пара файлов с англ. фразами.

6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot
Copy link
Contributor

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

@Kichikahunov
Copy link
Contributor Author

Kichikahunov commented Jul 28, 2019

@iliakan правки учитывал, но мог что-то упустить. Перевод статьи делал со своего форка, который делал пару месяцев назад, ранее, если были какие-то изменения в основном репозитории, то у меня появлялись конфликты, сейчас их не было 🤔

@Kichikahunov
Copy link
Contributor Author

@xcurveballx, ок, посмотрю

@Kichikahunov
Copy link
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Jul 28, 2019
@javascript-translate-bot javascript-translate-bot requested a review from a team July 28, 2019 08:28
@iliakan
Copy link
Member

iliakan commented Jul 28, 2019

Конфликты не поправишь?

Copy link
Member

@iliakan iliakan left a comment

Choose a reason for hiding this comment

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

Конфликты разреши плиз

@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels Jul 28, 2019
@javascript-translate-bot
Copy link
Contributor

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

@Kichikahunov
Copy link
Contributor Author

@iliakan, ок, сейчас сделаю

@Kichikahunov
Copy link
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Jul 28, 2019
@javascript-translate-bot javascript-translate-bot requested a review from a team July 28, 2019 15:36
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
6-data-storage/01-cookie/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot javascript-translate-bot removed the review needed Review needed, please approve or request changes label Jul 28, 2019
@javascript-translate-bot
Copy link
Contributor

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

@javascript-translate-bot javascript-translate-bot added the changes requested Waiting for changes and /done from PR author label Jul 28, 2019
@iliakan iliakan removed the partially reviewed Last review only took a part of the article. Full re-review (not just changes since last) required label Jul 28, 2019
@Kichikahunov
Copy link
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Jul 28, 2019
@javascript-translate-bot javascript-translate-bot requested a review from a team July 28, 2019 17:42
@iliakan
Copy link
Member

iliakan commented Jul 28, 2019

🚀

@javascript-translate-bot javascript-translate-bot merged commit fdc0ee8 into javascript-tutorial:master Jul 28, 2019
@iliakan
Copy link
Member

iliakan commented Jul 28, 2019

@Kichikahunov Поздравляю! Вы перевели последнюю главу учебника!
🍸 🍸 🍸 🍸 🍸
🎆 🎆 🎆 🎆 🎆
👍 👍 👍 👍 👍

@Kichikahunov Kichikahunov deleted the 6-data-storage/01-cookie branch July 29, 2019 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review needed Review needed, please approve or request changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants