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

[ru] improve Web/HTTP/Headers/Set-Cookie translation #22161

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

mnik01
Copy link

@mnik01 mnik01 commented Jul 4, 2024

Description

Translated set-cookie page to Russian locale by updating the .md file.

Motivation

There were not translated parts on the page and some repeated text blocks.

Additional details

Related issues and pull requests

@mnik01 mnik01 requested a review from a team as a code owner July 4, 2024 12:47
@mnik01 mnik01 requested review from saionaro and removed request for a team July 4, 2024 12:47
@github-actions github-actions bot added the l10n-ru Issues related to Russian content. label Jul 4, 2024
Copy link
Contributor

github-actions bot commented Jul 4, 2024

Preview URLs

Flaws (10)

URL: /ru/docs/Web/HTTP/Headers/Set-Cookie
Title: Set-Cookie
Flaw count: 10

  • macros:
    • /ru/docs/Web/HTTP/Basics_of_HTTP/Choosing_between_www_and_non-www_URLs does not exist but fell back to /en-US/docs/Web/HTTP/Basics_of_HTTP/Choosing_between_www_and_non-www_URLs
    • /ru/docs/Web/HTTP/Basics_of_HTTP/Evolution_of_HTTP does not exist but fell back to /en-US/docs/Web/HTTP/Basics_of_HTTP/Evolution_of_HTTP
    • /ru/docs/Web/HTTP/Protocol_upgrade_mechanism does not exist but fell back to /en-US/docs/Web/HTTP/Protocol_upgrade_mechanism
    • /ru/docs/Web/HTTP/Headers/X-Frame-Options does not exist but fell back to /en-US/docs/Web/HTTP/Headers/X-Frame-Options
    • /ru/docs/Web/Security/Practical_implementation_guides does not exist but fell back to /en-US/docs/Web/Security/Practical_implementation_guides
    • and 5 more flaws omitted
External URLs (5)

URL: /ru/docs/Web/HTTP/Headers/Set-Cookie
Title: Set-Cookie

(comment last updated: 2024-07-15 21:49:11)

@mnik01
Copy link
Author

mnik01 commented Jul 4, 2024

Стоит ли переводить

> **Warning:**

Как

> **Предупреждение:**

Или можно оставить слово warning как есть в английской версии?

@leon-win
Copy link
Member

leon-win commented Jul 4, 2024

Стоит ли переводить

> **Warning:**

Как

> **Предупреждение:**

Или можно оставить слово warning как есть в английской версии?

Да, конечно, нужно переводить. А почему сомнения?)

files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
@leon-win
Copy link
Member

leon-win commented Jul 4, 2024

Привет, @mnik01 !

Благодарю за желание улучшить MDN!

Я успел посмотреть часть изменений, предложил свои улучшения. И заметил, что перевод страницы не полностью сделан (напирмер, таблица про типы заголовков и далее примеры синтаксиса не дополнены в соотвествии со ��вежей версий оригинала). А судя по описанию PR'а планировалось полностью синхронизировать перевод всей страницы?

@leon-win leon-win changed the title Translated set-cookie page to Russian locale Jul 4, 2024
@mnik01
Copy link
Author

mnik01 commented Jul 5, 2024

Да, конечно, нужно переводить. А почему сомнения?)

На сайте эти заметки в маркдауне превращаются в плашки и Я не был уверен подхватит ли скрипт её если слово будет не warning.

Спасибо, за ответ.

files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
@mnik01
Copy link
Author

mnik01 commented Jul 5, 2024

@leon-win Я обновил файлик — синхронизовал с актуальной версией оригинала

Посмотрите, пожалуйста, всё ли ок.
И, кстати, как лучше всего перевести Compatibility notes? Заметки по совместимости?

files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
- В текущей спецификация начальная точка в имени хоста игнорируется (.example.com)
- Cookie будут отправляться также на поддомены указанного хоста
- Указывать несколько хостов недопустимо.
Имя куки (`<cookie-name>`) может содержать любые символы US-ASCII, за исключением: управляющих символов (символы {{Glossary("ASCII")}} от 0 до 31 и символ ASCII 127) или разделительных символов (пробел, табуляция и символы: `( ) < > @ , ; : \ " / [ ] ? = { }`).
Copy link
Contributor

Choose a reason for hiding this comment

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

[markdownlint] reported by reviewdog 🐶
MD046/code-block-style Code block style [Expected: fenced; Actual: indented]

files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
files/ru/web/http/headers/set-cookie/index.md Outdated Show resolved Hide resolved
mnik01 and others added 8 commits July 6, 2024 19:59
Co-authored-by: Leonid Vinogradov <win.leon@protonmail.com>
Co-authored-by: Leonid Vinogradov <win.leon@protonmail.com>
Co-authored-by: Leonid Vinogradov <win.leon@protonmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@leon-win
Copy link
Member

leon-win commented Jul 7, 2024

Да, конечно, нужно переводить. А почему сомнения?)

На сайте эти заметки в маркдауне превращаются в плашки и Я не был уверен подхватит ли скрипт её если слово будет не warning.

Да, yari (движок, который обрабатывает документы MDN) производит множество манипуляций с разметко��, в том числе создаёт такие плашки. Он действительно "заточен" на конкретные слова (с сочетаниями символов), в данном случае дял русского языка > **Предупреждение:** будет преобразовано в специальную разметку.

Если я правильно понял (раз возникают такие опасения + много автоматических замечаний по оформлению от бота), то ты ведёшь работу "вслепую", то есть не запуская MDN локально? Если да, то это не очень хорошо( Использовать Github веб-интерфейс удобно только для мелких изменений.

А если переводить документ полностью, то лучше развернуть проект локально, для этого нужно слконировать оба репозитория (content и translated-content). Как это сделать описано в https://github.com/mdn/content/blob/main/CONTRIBUTING.md#forking-and-cloning-the-repository (начиная с этого раздела и далее) и https://github.com/mdn/translated-content/blob/main/CONTRIBUTING.md. А также есть наш небольшой гайд по особенностям перевода именно на русский язык (https://github.com/mdn/translated-content/blob/main/docs/ru/translation-guide.md).

Посмотрите, пожалуйста, всё ли ок.

Да, я постараюсь найти время и посмотреть 👌

И, кстати, как лучше всего перевести Compatibility notes? Заметки по совместимости?

Я думаю, что лучше "Замечания по совместимости".

@mnik01
Copy link
Author

mnik01 commented Jul 8, 2024

Если я правильно понял , то ты ведёшь работу "вслепую"

Да, это так.

Гайд по особенностям перевода именно на русский я читал, а за наводку на то как локально развернуть проект — спасибо 👍

Попробую поднять когда (если) будет время

Copy link
Member

@leon-win leon-win left a comment

Choose a reason for hiding this comment

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

Проверил ещё часть документа и внёс правки, дошёл до описания атрибута Domain=<domain-value>.

Но такими темпами очень долго будет(( Слишком много неточностей и проблем с форматированием, часть описания продублирована была (про кодирование), не обновлён синтаксис и т.д.

Нужно более внимательно подойти к переводу, и делать это в IDE с запуском локального сервера. А также обращать внимание на сообщения от бота github-actions об ошибка форматирвоания. С ними влить всё равно не получится))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ru Issues related to Russian content.
2 participants