feature/issues_v1 #25

Open
DmitrySheyko wants to merge 12 commits from feature/issues_v1 into develop
Member

Dmitry Sheyko.
Pull Request 1:

  • cозданы доменные сущности для функционала Issue;
  • созданы аналогичные сущности Json;
  • содан changelog создания таблиц для сущностей функционала Issue.
Dmitry Sheyko. Pull Request 1: - cозданы доменные сущности для функционала Issue; - созданы аналогичные сущности Json; - содан changelog создания таблиц для сущностей функционала Issue.
DmitrySheyko added 2 commits 2023-01-19 18:47:23 +03:00
upagge added 1 commit 2023-01-20 18:36:37 +03:00
upagge reviewed 2023-01-20 18:48:18 +03:00
upagge left a comment
Owner

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

В целом все хорошо. Однако, нам для работы уведомлений не обязательно сохранять всю информацию по Issue, которая поступает. Стоит ограничиться только набором необходимых для работы полей. Я удалил лишние классы, а также лишние поля в Issue. Теперь нужно привести схему к новой доменной модели. Исправлять Json классы не нужно, там все хорошо.
@ -0,0 +1,6 @@
package dev.struchkov.bot.gitlab.context.domain;
public enum IssueType {
ISSUE,
Owner

Нужно писать в одном стиле. Сейчас даже в рамках одного ПР стиль отличается. За образец предлагаю взять MergeRequestState

Нужно писать в одном стиле. Сейчас даже в рамках одного ПР стиль отличается. За образец предлагаю взять MergeRequestState
Author
Member

Правки внес.

Правки внес.
@ -0,0 +34,4 @@
*
* @author Dmitry Sheyko [17.01.2023]
*/
Owner

Лишний перенос строки, и комментарий непонятно что означает, почему это стоит учитывать и почему этот комментарий тут? Пагинация передается и настраивается. Комментарий стоит удалить.

Лишний перенос строки, и комментарий непонятно что означает, почему это стоит учитывать и почему этот комментарий тут? Пагинация передается и настраивается. Комментарий стоит удалить.
Author
Member

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

Правки внес. Комментарий думал оставить временно, чтобы не забыть осоенность которую нашел в документации. Привычка везде оставлять напоминания.
@ -0,0 +4,4 @@
import java.util.Set;
public class IssueRequestParser {
Owner

Пр должен иметь логическое заключение, это явно лишний клас. По хорошему, нужно удалить.

Пр должен иметь логическое заключение, это явно лишний клас. По хорошему, нужно удалить.
Author
Member

Удалил.
Логику понял. Внчачале работы не знал с чего начать и создал класс:)

Удалил. Логику понял. Внчачале работы не знал с чего начать и создал класс:)
@ -0,0 +3,4 @@
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-4.17.xsd">
<changeSet id="2023-01-19-create-table-milestone" author="Dmitry Sheyko">
Owner

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

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

ChangeSet доработал

ChangeSet доработал
@ -0,0 +44,4 @@
</createTable>
</changeSet>
<changeSet id="create-table-issue" author="Dmitry Sheyko">
Owner

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

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

Даты внес

Даты внес
@ -0,0 +179,4 @@
</changeSet>
<changeSet id="create-table-issue_labels" author="Dmitry Sheyko">
<createTable tableName="issue_labels">
Owner

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

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

Переименовал

Переименовал
@ -0,0 +184,4 @@
<constraints nullable="false" foreignKeyName="fk_issue_labels_issue_id"
references="issue(id)" deleteCascade="true"/>
</column>
<column name="labels" type="varchar(255)">
Owner

labels -> label

labels -> label
Author
Member

Сделано

Сделано
@ -0,0 +15,4 @@
*/
@Data
public class IssueJson {
Owner

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

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

Понятно

Понятно
@ -0,0 +74,4 @@
private Integer discussionLocked;
@JsonProperty("issue_type")
private String issueType; //TODO выяснить зачем дублирует поле type
Owner

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

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

Удалено

Удалено
@ -0,0 +5,4 @@
/**
* @author Dmitry Sheyko [17.01.2023]
*/
Owner

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

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

Переносы удалил

Переносы удалил
@ -0,0 +8,4 @@
*/
@Data
public class ReferencesJson {
Owner

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

@Data
public class ReferencesJson {

    @JsonProperty("full")
    private String fullReference;
    
}
Код стайл в данном проекте это перенос строки в начале класса и в конце. Пример ``` @Data public class ReferencesJson { @JsonProperty("full") private String fullReference; } ```
Author
Member

Стиль исправил

Стиль исправил
upagge requested review from upagge 2023-01-20 18:48:34 +03:00
upagge added the
feature
label 2023-01-20 18:48:39 +03:00
upagge added this to the Версия бота 2.0 project 2023-01-20 18:49:39 +03:00
upagge added this to the Sprint 1 milestone 2023-01-20 18:49:42 +03:00
DmitrySheyko was assigned by upagge 2023-01-20 18:49:52 +03:00
upagge started working 2023-01-20 18:51:55 +03:00
DmitrySheyko added 1 commit 2023-01-22 00:56:46 +03:00
upagge stopped working 2023-01-28 13:24:51 +03:00
1 week
DmitrySheyko added 1 commit 2023-02-03 20:22:57 +03:00
DmitrySheyko added 7 commits 2023-03-11 13:13:37 +03:00
# Conflicts:
#	bot-context/src/main/java/dev/struchkov/bot/gitlab/context/domain/entity/Issue.java
#	bot-context/src/main/java/dev/struchkov/bot/gitlab/context/utils/Icons.java
#	bot-core/src/main/java/dev/struchkov/bot/gitlab/core/service/parser/IssueParser.java
#	bot-core/src/main/java/dev/struchkov/bot/gitlab/core/service/parser/forktask/GetAllIssueForProjectTask.java
#	bot-core/src/main/java/dev/struchkov/bot/gitlab/core/service/parser/forktask/GetSingleIssueTask.java
#	gitlab-app/src/main/java/dev/struchkov/bot/gitlab/scheduler/SchedulerService.java
#	gitlab-app/src/main/resources/application.yml
#	gitlab-app/src/main/resources/liquibase/v.1.0.0/2023-01-19-create-tables-for-issue.xml
#	telegram-bot/src/main/java/dev/struchkov/bot/gitlab/telegram/service/notify/DeleteFromAssigneesOfIssueNotifyGenerator.java
#	telegram-bot/src/main/java/dev/struchkov/bot/gitlab/telegram/service/notify/DescriptionIssueNotifyGenerator.java
#	telegram-bot/src/main/java/dev/struchkov/bot/gitlab/telegram/service/notify/DueDateIssueNotifyGenerator.java
#	telegram-bot/src/main/java/dev/struchkov/bot/gitlab/telegram/service/notify/NewIssueNotifyGenerator.java
#	telegram-bot/src/main/java/dev/struchkov/bot/gitlab/telegram/service/notify/StatusIssueNotifyGenerator.java
#	telegram-bot/src/main/java/dev/struchkov/bot/gitlab/telegram/service/notify/TitleIssueNotifyGenerator.java
#	telegram-bot/src/main/java/dev/struchkov/bot/gitlab/telegram/service/notify/TypeIssueNotifyGenerator.java
This pull request has changes conflicting with the target branch.
  • bot-core/src/main/java/dev/struchkov/bot/gitlab/core/config/properties/GitlabProperty.java
  • gitlab-app/src/main/java/dev/struchkov/bot/gitlab/scheduler/SchedulerService.java
  • gitlab-app/src/main/resources/application.yml

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/issues_v1:feature/issues_v1
git checkout feature/issues_v1
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Telegram-Bots/gitlab-notification#25
No description provided.