Skip to content

feat: Библиотека и тесты переведены на английский язык - #79

Merged
vbondarevsky merged 4 commits into
vbondarevsky:masterfrom
slaxxals:master
Sep 10, 2021
Merged

vbondarevsky merged 4 commits into
vbondarevsky:masterfrom
slaxxals:master

Conversation

@slaxxals

@slaxxals slaxxals commented Sep 7, 2021 •

Copy link
Copy Markdown
Contributor

Для использования библиотеки при разработке в конфигурациях с Вариантом встроенного языка = English.

@slaxxals slaxxals changed the title feat: Перевести библиотеку на английский язык для использования при р… feat: Перевести библиотеку на английский язык Sep 7, 2021
@slaxxals slaxxals changed the title feat: Перевести библиотеку на английский язык feat: Библиотека и тесты переведены на английский язык Sep 8, 2021
@BlizD

BlizD commented Sep 8, 2021 •

Copy link
Copy Markdown
Contributor

А зачем и код переводить на английский? Возможно достаточно только сообщения на английский перевести?

У нас не используется английский код, поэтому нам это не требуется и будет только мешать.

@slaxxals

slaxxals commented Sep 8, 2021 •

Copy link
Copy Markdown
Contributor Author

У нас не используется английский код, поэтому нам это не требуется и будет только мешать.

У нас используется английский вариант встроенного языка. Некоторые разработчики вообще русский язык не знают. Также мы используем БСП на английском (SSL). Я уверен, что таких, как у нас команд разработчиков, не одна, а больше. Поэтому разумным было бы портировать библиотеку полностью на английский язык. И при любом изменении в основной ветке на русском языке - дублировать их на английском. На данный момент переведен весь код и все тесты. Я начал переводить комментарии к методам, а потом и до README дело дойдет.

Если же вам это не требуется и будет мешать, то можно сделать отдельную ветку "master-en", например.

@BlizD

BlizD commented Sep 8, 2021

Copy link
Copy Markdown
Contributor

Если же вам это не требуется и будет мешать, то можно сделать отдельную ветку "master-en", например.

Да, этот вариант нам больше подходит

@slaxxals

slaxxals commented Sep 8, 2021

Copy link
Copy Markdown
Contributor Author

Да, этот вариант нам больше подходит

Отлично! Попробую создать в моем форке отдельную ветку и сделать PR, в котором будут исходники конфигурации полностью на английском языке. Если не получится, то прошу вас сделать эту ветку master-en, а я в нее уже отправлю PR.

@BlizD

BlizD commented Sep 8, 2021

Copy link
Copy Markdown
Contributor

Решение за @vbondarevsky я лишь высказал свое мнение

@slaxxals

slaxxals commented Sep 8, 2021

Copy link
Copy Markdown
Contributor Author

Решение за @vbondarevsky я лишь высказал свое мнение

Хорошо, тогда призываю Владимира @vbondarevsky к обсуждению.

Какой вариант существования библиотеки на английском языке наиболее приемлемый?

  1. Одна ветка master, с исходниками одной конфигурации, в которой продублированы переведенные объекты ОМ.КоннекторHTTP и Обработка.Тесты. (этот вариант сейчас присутствует в текущем PR)
  2. Новая ветка master-en, с исходниками одной конфигурации, в которой присутствуют только переведенные объекты. Исходных объектов из конфигурации ветки master в ней не будет.
  3. Библиотека переведенная на английский язык не нужна в этом репозитории.

@slaxxals

slaxxals commented Sep 9, 2021

Copy link
Copy Markdown
Contributor Author

Нашел issue для этого PR - вот #46

@vbondarevsky

Copy link
Copy Markdown
Owner

Решение за @vbondarevsky я лишь высказал свое мнение

Хорошо, тогда призываю Владимира @vbondarevsky к обсуждению.

Какой вариант существования библиотеки на английском языке наиболее приемлемый?

1. Одна ветка master, с исходниками одной конфигурации, в которой продублированы переведенные объекты ОМ.КоннекторHTTP и Обработка.Тесты. (этот вариант сейчас присутствует в текущем PR)

2. Новая ветка master-en, с исходниками одной конфигурации, в которой присутствуют только переведенные объекты. Исходных объектов из конфигурации ветки master в ней не будет.

3. Библиотека переведенная на английский язык не нужна в этом репозитории.

Думаю лучше так:

  • в папке src сделать две подпапки ru и en. В ru переместить текущие исходники, а в en вашу версию с переводом
  • отдельное README для английского и в основном сделать ссылку в начале

@slaxxals

slaxxals commented Sep 9, 2021

Copy link
Copy Markdown
Contributor Author

@vbondarevsky Отлично!

Тогда есть еще один момент:

Так как библиотека периодически изменяется, то с появлением новых изменений на русском необходимо вносить эти же изменения и в английский вариант, только уже переведенные. Как лучше оформлять такие коммиты?

@vbondarevsky

Copy link
Copy Markdown
Owner

@vbondarevsky Отлично!

Тогда есть еще один момент:

Так как библиотека периодически изменяется, то с появлением новых изменений на русском необходимо вносить эти же изменения и в английский вариант, только уже переведенные. Как лучше оформлять такие коммиты?

Это уже непринципиально: один коммит сразу в 2 версии или на каждую свой отдельный. Переносить изменения лучше покоммитно, т.е. одно изменение в русской версии = одно изменение в английской

@slaxxals

slaxxals commented Sep 9, 2021

Copy link
Copy Markdown
Contributor Author

Понятно, тогда раскидываю исходники на 2 директории и продолжаю перевод комментариев и README

@vbondarevsky

Copy link
Copy Markdown
Owner

Понятно, тогда раскидываю исходники на 2 директории и продолжаю перевод комментариев и README

Только текущие файлы лучше мувнуть (git mv) чтобы не было кучи новых изменений

@slaxxals slaxxals reopened this Sep 9, 2021
@slaxxals

Copy link
Copy Markdown
Contributor Author

@vbondarevsky библиотека переведена на английский полностью, исходники разбиты по директориям.

Прошу рассмотреть PR.

@vbondarevsky vbondarevsky left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@slaxxals
Пока волью как есть. Замечания предлагаю поправить отдельным PR.

Английский вариант особо подробно не смотрел, но в целом выглядит ОК

Comment thread README.md

**Коннектор** - это "Requests" для мира 1С.

Бибилиотека полностью переведена на английский язык ([см. здесь](./README-EN.md)).

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Думаю эту фразу лучше написать на английском для тех кто не знает русский

Comment thread README-EN.md
[![Stars](https://img.shields.io/github/stars/vbondarevsky/Connector.svg?label=Github%20%E2%98%85&a)](https://github.com/vbondarevsky/Connector/stargazers)
[![Release](https://img.shields.io/github/tag/vbondarevsky/Connector.svg?label=Last%20release&a)](https://github.com/vbondarevsky/Connector/releases)

# Connector: handy HTTP-client for 1C Enterprise 8 platform

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

1C:Enterprise

@@ -0,0 +1,3083 @@
// Коннектор: удобный HTTP-клиент для 1С:Предприятие 8

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Connector: handy HTTP-client for 1C:Enterprise 8 platform

// e-mail: vbondarevsky@gmail.com
// Версия: 2.3.1
//
// Требования: платформа 1С версии 8.3.10 и выше

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

А это?


#Region CommonUseMethods

// Отправляет GET запрос

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Описание функций тоже потом?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Да, в ближайшем комите

Comment thread README-EN.md
- `Basic`, `Digest` and `AWS4-HMAC-SHA256` authentification
- Auto resolution of redirections
- Cookies setting and getting
- Session with state persistance across requests (cookies, authentification и пр.)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

и пр.

Comment thread README-EN.md
- And much more

## Requirements
- 1C Enterprise platform version **8.3.10** and higher.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

1C Enterprise -> 1C:Enterprise
здесь и в других местах

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Есть ли версия библиотеки в англоязычном синтаксисе?

3 participants