Struchkov Mark upagge
upagge pushed to develop at Haiti/haiti-utils-field-constants 2023-01-23 14:25:32 +03:00
4563ce2c4e v.0.0.6
e759449bc6 Merge remote-tracking branch 'tsc/develop' into develop
9f41dde9ec fix package on codegen classes
6d707cf8c8 NextRelease
e7420705b8 release-0.4.0
Compare 10 commits »
upagge created branch develop in Haiti/haiti-utils-field-constants 2023-01-23 14:25:32 +03:00
upagge pushed tag v.0.0.2 to Haiti/haiti-utils-field-constants 2023-01-23 14:25:32 +03:00
upagge pushed tag v.0.0.3 to Haiti/haiti-utils-field-constants 2023-01-23 14:25:32 +03:00
upagge pushed tag v.0.0.4 to Haiti/haiti-utils-field-constants 2023-01-23 14:25:32 +03:00
upagge created repository Haiti/haiti-utils-field-constants 2023-01-23 14:22:19 +03:00
upagge opened issue Telegram-Bots/gitlab-notification#28 2023-01-21 01:37:45 +03:00
Многопользовательский режим
upagge commented on issue Telegram-Bots/gitlab-notification#9 2023-01-21 01:34:46 +03:00
Определение размера MR

Или можно вытаскивать эту информацию только перед непосредственно уведомлением. Чтобы не вытаскивать эти…

upagge commented on issue Telegram-Bots/gitlab-notification#9 2023-01-21 01:33:31 +03:00
Определение размера MR

Можно попробовать добовлять эту информацию при последующем обходе старых МР.

upagge opened issue Telegram-Bots/gitlab-notification#27 2023-01-21 01:32:18 +03:00
Определение статусов ревьюверов
upagge opened issue Telegram-Bots/gitlab-notification#26 2023-01-21 01:19:39 +03:00
Возможность указывать рабочее время
upagge commented on pull request Telegram-Bots/gitlab-notification#25 2023-01-20 18:48:18 +03:00
feature/issues_v1

Код стайл в данном проекте это перенос строки в начале класса и в конце.

upagge commented on pull request Telegram-Bots/gitlab-notification#25 2023-01-20 18:48:18 +03:00
feature/issues_v1

В целом все хорошо. Однако, нам для работы уведомлений не обязательно сохранять всю информацию по Issue, которая поступает. Стоит ограничиться только набором необходимых для работы полей. Я удалил лишние классы, а также лишние поля в Issue. Теперь нужно привести схему к новой доменной модели. Исправлять Json классы не нужно, там все хорошо.

upagge commented on pull request Telegram-Bots/gitlab-notification#25 2023-01-20 18:48:18 +03:00
feature/issues_v1

Лишние переносы нужно удалить во всех классах

upagge commented on pull request Telegram-Bots/gitlab-notification#25 2023-01-20 18:48:18 +03:00
feature/issues_v1

Вот этот класс не обязательно упращать. Можно и даже нужно оставить его в таком виде.

upagge commented on pull request Telegram-Bots/gitlab-notification#25 2023-01-20 18:48:18 +03:00
feature/issues_v1

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

upagge commented on pull request Telegram-Bots/gitlab-notification#25 2023-01-20 18:48:18 +03:00
feature/issues_v1

labels -> label

upagge commented on pull request Telegram-Bots/gitlab-notification#25 2023-01-20 18:48:18 +03:00
feature/issues_v1

В целях следования единому стилю, нужно переименовать на issue_label

upagge commented on pull request Telegram-Bots/gitlab-notification#25 2023-01-20 18:48:18 +03:00
feature/issues_v1

пропущена дата в id

upagge commented on pull request Telegram-Bots/gitlab-notification#25 2023-01-20 18:48:18 +03:00
feature/issues_v1

Нужно актуализировать схему согласно упращенной доменной модели