Struchkov Mark upagge
upagge pushed tag v.0.0.2 to Haiti/haiti-utils-field-constants 2023-01-23 14:25:32 +03:00
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.1 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

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

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

labels -> label

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

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

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

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