search

pf03 / newBot

BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Управление ботом #4

Closed KateBushueva closed 3 years ago

KateBushueva commented 3 years ago

Пропиши, пожалуйста, в ридми, управление ботом:

KateBushueva commented 3 years ago

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

KateBushueva commented 3 years ago

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

pf03 commented 3 years ago

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

Да, действительно у меня предусмотрено только количество повторов для конкретного пользователя (repeatNumber) Для всех пользователей сразу я не прописывал, значит неправильно понял задание

pf03 commented 3 years ago

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

добавил поле defaultRepeatNumber в config.json

pf03 commented 3 years ago

про приоритет логгирования теперь увидела в конфиге, но все равно лучше в ридми добавить.

добавил

И допустимые значение тоже надо прописать (это, кстати, намек, что должна быть проверка данных из файла на корректность)

добавил проверку на корректность для repeatNumber, log.minLevel. Остальные поля проверяет сам aeson при парсинге, по крайней мере по типам

KateBushueva commented 3 years ago

По поводу ридми осталось поправить:

pf03 commented 3 years ago

По поводу ридми осталось поправить:

  • пункт 2: файл уже присутствует, не нужно вводить в заблуждение;

Это я имел в виду, что если exe-шник запускать отдельно (a не stack ghci как обычно) , то к нему в папку нужно скопировать файл конфига. Ну не проблема удалить

Sergey111991KV commented 3 years ago

отлично)