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.

Sign up for GitHub

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

Jump to bottom

WIP: Перевести разработку в EDT и задействовать LangTool #102

Closed
wants to merge 10 commits into from
Closed

WIP: Перевести разработку в EDT и задействовать LangTool #102

wants to merge 10 commits into from

Conversation

zeegin
Copy link
Contributor

@zeegin zeegin commented Nov 20, 2021 •
edited
Loading

Основная задача:

  • Перевести разработку в EDT и задействовать LangTool #97
  • Проконтроллировать полноту перевода, отсутствие не переведенных фрагментов
  • Перенастроить CI для новой структуры файлов для EDT
  • Добиться прохождения всех тестов на собранной версии
    • Test_ReceivingReleasesListFrom1CSite
    • Test_DigestAuth
    • Test_GetLoopedRedirect
    • Тест_AuthentificationAWS4_HMAC_SHA256
    • Test_RetryBasedOnHeaderRetryAfterDate
    • Test_RetryBasedOnHeaderRetryAfterDuration
    • Test_RetryAfterError502
    • Test_RetryAfterConnectionError
    • Test_Ping
  • Добиться прохождения всех тестов из прошлой сборки английской версии, регресс

Дополнительные задачи:

Проблемы:

  • Теряется блейм для формы и метаданных. Нужен ли он? Блейм для модуля тестов и модуля коннектора удалось сохранить.
  • Версия совместимости поднята до 8.3.14. Без нее есть проблема с переводом методов из библиотеки платформы. StrStartsWith.

@zeegin
feat: Перевод разработки в EDT
778d5d0 
- Изменена структура папок
- Удален проект с английским кодом
@zeegin
feat: Добавление английского варианта проекта
946afdd
@zeegin
feat: Конструкторы аутентификации
71f3ddb 
- исправление проблем типизации для перевода

vbondarevsky#101
@zeegin
fix: Работа до перевода
521139d 
- устранение проблем типизации кода
- расстановка тегов непереводимых строковых литералов
@zeegin
translate: Сборка словаря переводов
43351f6
@zeegin
feat: Сборка переведенной конфигурации
5cfcccf 
ОШИБКИ:
- Test_SendCookies
- Тест_POST_MultipartFormData_FileOnly
- Тест_POST_MultipartFormData_FilesAndFormFields
- Test_VerifyOfRestoringUnsupportedValuesTypes

НЕ ПРОВЕДЕННЫЕ ТЕСТЫ:
- Test_ConnectionViaProxy
- Test_ReceivingReleasesListFrom1CSite
- Тест_AuthentificationAWS4_HMAC_SHA256
- Test_RetryBasedOnHeaderRetryAfterDate
- Test_RetryBasedOnHeaderRetryAfterDuration
- Test_RetryAfterError502
- Test_RetryAfterConnectionError
- Test_Ping
@zeegin
docs: Минимальная вводная по переводу
be12f4c
@zeegin
Copy link
Contributor Author

zeegin commented Nov 20, 2021

Синхронный перевод выглядит так:
image

@zeegin
translate: Добавление типизации и переводов
2cd123c
@zeegin
fis: Основные тесты прошли
7efcf9c 
- Версия совместимости поднята до 8.3.14
- Исправления типизации в отнеошении литералов
@zeegin
Copy link
Contributor Author

zeegin commented Nov 20, 2021

@slaxxals @vbondarevsky
Проверьте пожалуйста сможете ли вы такой проект собрать и запустить.

zeegin
zeegin commented Nov 20, 2021
View reviewed changes
Copy link
Contributor Author

@zeegin zeegin left a comment

Choose a reason for hiding this comment

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

Нет перевода шапки модуля и пока нет возможности ее запрограммировать плагином.

vbondarevsky
vbondarevsky reviewed Nov 20, 2021
View reviewed changes
src/Connector.En/src/Configuration/Configuration.mdo
<dataLockControlMode>Managed</dataLockControlMode>
<objectAutonumerationMode>NotAutoFree</objectAutonumerationMode>
<modalityUseMode>DontUse</modalityUseMode>
<compatibilityMode>8.3.14</compatibilityMode>
Copy link
Owner

Choose a reason for hiding this comment

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

До EDT была версия: 8.3.10

@slaxxals
Copy link
Contributor

slaxxals commented Dec 5, 2021 •
edited
Loading

@slaxxals @vbondarevsky Проверьте пожалуйста сможете ли вы такой проект собрать и запустить.

@zeegin В ЕДТ только начинаю работать, проекты Connector, Connector.En открыл, добавил новые базы для каждого из проектов запустил тесты, все работает (кроме редиректов). Пока в процессе изучения как пользоваться переводом.

Пришлось понизить версию с 8.3.20 на 8.3.19, пока нет возможности работать с 8.3.20.

@zeegin
Copy link
Contributor Author

zeegin commented Dec 5, 2021

Пока не все тесты проходят. Я еще адрес сервера тестирования подменил, который @vbondarevsky развернул. Там сильно лучше.

@zeegin
Copy link
Contributor Author

zeegin commented Dec 5, 2021

Надо разобраться со StrStartsWith есть подозрение что и в русской версии на 8.3.10 коннектор не работает.

@zeegin
Copy link
Contributor Author

zeegin commented Dec 5, 2021

По lang tool дока здесь
https://its.1c.ru/db/edtplugins#bookmark:languagetool:Introduction

@marmyshev
Copy link

По lang tool дока здесь
https://its.1c.ru/db/edtplugins#bookmark:languagetool:Introduction

Документацию лучше использовать встроенную в EDT - в меню: Help -> Help contents -> 1C:Language Tool - она там сильно актуальней.

@zeegin
feat: Задействование connectorhttp.ru вместо httpbin.org
d6371f6 
- удалена непереведенная ветка тестов
@leemuar
Copy link
Collaborator

leemuar commented Dec 29, 2023

Активности в PR нет уже 2 года, закрываю. При необходимости откройте новый PR

@leemuar leemuar closed this Dec 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vbondarevsky vbondarevsky vbondarevsky left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@zeegin @slaxxals @marmyshev @leemuar @vbondarevsky