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

Issues 55:Изменен файл настроек, добавлено логирование в ЖР #56

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

Arcius7012
Copy link

Добрый день!
Делал для себя, буду рад если пригодится в основном проекте

@Stepa86
Copy link
Collaborator

Stepa86 commented May 24, 2020

Изменение файла параметров - ок, а вот запись в журнал сомнительна.

При полностью правильном процессе - АПК создается/разворачивается, затем в нее подгружается инфа о проверяемой конфе, выполняется проверка, выгружается результат и база дропается.

Получить информацию о процессе можно из спец. файла лога - если это написать или из лога раннера, который в stdout пишет, если писать Сообщить( в коде.

Я бы наверно заменил вывод в журнал на аналог логера:

  1. Новый параметр УровеньЛогирования
  2. Вывод через Сообщить в зависимости от уровня

@otymko
Copy link
Owner

otymko commented Jun 25, 2020

@Arcius7012 Поддерживаю комментарии @Stepa86 выше. По хорошему должен быть общий метод логирования с уровнями. Очень частный вариант использования 1С: АПК + acc_export -> запустили проверку, после анализа и экспорта база удаляется.

@otymko otymko marked this pull request as draft June 26, 2020 13:32
@Arcius7012
Copy link
Author

Ок, сегодня-завтра сделаю

@Arcius7012
Copy link
Author

Уровень логирования добавил в обработку и настройки, я только не понял как правильно разложить обработку acc-export.epf в каталог "src\acc-export"

@Arcius7012 Arcius7012 closed this Jul 6, 2020
@Arcius7012 Arcius7012 reopened this Jul 6, 2020
@Arcius7012
Copy link
Author

Разобрался, настроил прекоммит

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