Telegram-Bots/gitlab-notification
4
1
Fork 0
Code Issues 21 Pull Requests Packages Projects 2 Releases 1 Activity

feature/issues_v1 #25

Closed
DmitrySheyko wants to merge 12 commits from feature/issues_v1 into develop
pull from: feature/issues_v1
merge into: Telegram-Bots:develop
Conversation 1 Commits 12 Files Changed 44 +2056 -5
DmitrySheyko commented 2023-01-19 18:47:22 +03:00
Member
Copy Link

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
Feat: создана сущность Issue и вспомогательные, приступаю к миграциям БД. f146165024
Feat: создан changelog для таблиц сущности issue. b0fb70e150
upagge added 1 commit 2023-01-20 18:36:37 +03:00
Упростил сущность Issue 331ddd7314
upagge reviewed 2023-01-20 18:48:18 +03:00
upagge left a comment
Owner
Copy Link

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

В целом все хорошо. Однако, нам для работы уведомлений не обязательно сохранять всю информацию по Issue, которая поступает. Стоит ограничиться только набором необходимых для работы полей. Я удалил лишние классы, а также лишние поля в Issue. Теперь нужно привести схему к новой доменной модели. Исправлять Json классы не нужно, там все хорошо.
🚀 1
bot-context/src/main/java/dev/struchkov/bot/gitlab/context/domain/IssueType.java Outdated
@@ -0,0 +1,6 @@
package dev.struchkov.bot.gitlab.context.domain;
public enum IssueType {
ISSUE,
upagge commented 2023-01-20 18:37:49 +03:00
Owner
Copy Link

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

Нужно писать в одном стиле. Сейчас даже в рамках одного ПР стиль отличается. За образец предлагаю взять MergeRequestState
DmitrySheyko commented 2023-01-21 12:41:51 +03:00
Author
Member
Copy Link

Правки внес.

Правки внес.
bot-context/src/main/java/dev/struchkov/bot/gitlab/context/domain/entity/Issue.java
@@ -0,0 +34,4 @@
*
* @author Dmitry Sheyko [17.01.2023]
*/
upagge commented 2023-01-20 18:39:05 +03:00
Owner
Copy Link

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

Лишний перенос строки, и комментарий непонятно что означает, почему это стоит учитывать и почему этот комментарий тут? Пагинация передается и настраивается. Комментарий стоит удалить.
DmitrySheyko commented 2023-01-21 12:44:24 +03:00
Author
Member
Copy Link

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

Правки внес. Комментарий думал оставить временно, чтобы не забыть осоенность которую нашел в документации. Привычка везде оставлять напоминания.
bot-core/src/main/java/dev/struchkov/bot/gitlab/core/service/parser/IssueRequestParser.java Outdated
@@ -0,0 +4,4 @@
import java.util.Set;
public class IssueRequestParser {
upagge commented 2023-01-20 18:40:47 +03:00
Owner
Copy Link

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

Пр должен иметь логическое заключение, это явно лишний клас. По хорошему, нужно удалить.
DmitrySheyko commented 2023-01-21 12:46:51 +03:00
Author
Member
Copy Link

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

Удалил. Логику понял. Внчачале работы не знал с чего начать и создал класс:)
gitlab-app/src/main/resources/liquibase/v.2.0.0/2023-01-19-create-tables-for-issue.xml Outdated
@@ -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">
upagge commented 2023-01-20 18:41:07 +03:00
Owner
Copy Link

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

Нужно актуализировать схему согласно упращенной доменной модели
DmitrySheyko commented 2023-01-22 00:15:28 +03:00
Author
Member
Copy Link

ChangeSet доработал

ChangeSet доработал
gitlab-app/src/main/resources/liquibase/v.2.0.0/2023-01-19-create-tables-for-issue.xml
@@ -0,0 +44,4 @@
</createTable>
</changeSet>
<changeSet id="create-table-issue" author="Dmitry Sheyko">
upagge commented 2023-01-20 18:41:35 +03:00
Owner
Copy Link

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

пропущена дата в id
DmitrySheyko commented 2023-01-22 00:17:12 +03:00
Author
Member
Copy Link

Даты внес

Даты внес
gitlab-app/src/main/resources/liquibase/v.2.0.0/2023-01-19-create-tables-for-issue.xml Outdated
@@ -0,0 +179,4 @@
</changeSet>
<changeSet id="create-table-issue_labels" author="Dmitry Sheyko">
<createTable tableName="issue_labels">
upagge commented 2023-01-20 18:42:52 +03:00
Owner
Copy Link

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

В целях следования единому стилю, нужно переименовать на issue_label
DmitrySheyko commented 2023-01-22 00:15:40 +03:00
Author
Member
Copy Link

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

Переименовал
gitlab-app/src/main/resources/liquibase/v.2.0.0/2023-01-19-create-tables-for-issue.xml Outdated
@@ -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)">
upagge commented 2023-01-20 18:43:15 +03:00
Owner
Copy Link

labels -> label

labels -> label
DmitrySheyko commented 2023-01-22 00:15:51 +03:00
Author
Member
Copy Link

Сделано

Сделано
gitlab-sdk/src/main/java/dev/struchkov/bot/gitlab/sdk/domain/IssueJson.java Outdated
@@ -0,0 +15,4 @@
*/
@Data
public class IssueJson {
upagge commented 2023-01-20 18:44:26 +03:00
Owner
Copy Link

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

Вот этот класс не обязательно упращать. Можно и даже нужно оставить его в таком виде.
DmitrySheyko commented 2023-01-22 00:16:57 +03:00
Author
Member
Copy Link

Понятно

Понятно
gitlab-sdk/src/main/java/dev/struchkov/bot/gitlab/sdk/domain/IssueJson.java Outdated
@@ -0,0 +74,4 @@
private Integer discussionLocked;
@JsonProperty("issue_type")
private String issueType; //TODO выяснить зачем дублирует поле type
upagge commented 2023-01-20 18:44:03 +03:00
Owner
Copy Link

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

TODO можно удалить, скоре всего это просто оставлено для обратной совместимости со старым апи
DmitrySheyko commented 2023-01-22 00:16:06 +03:00
Author
Member
Copy Link

Удалено

Удалено
gitlab-sdk/src/main/java/dev/struchkov/bot/gitlab/sdk/domain/MilestoneStateJson.java
@@ -0,0 +5,4 @@
/**
* @author Dmitry Sheyko [17.01.2023]
*/
upagge commented 2023-01-20 18:45:06 +03:00
Owner
Copy Link

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

Лишние переносы нужно удалить во всех классах
DmitrySheyko commented 2023-01-22 00:16:38 +03:00
Author
Member
Copy Link

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

Переносы удалил
gitlab-sdk/src/main/java/dev/struchkov/bot/gitlab/sdk/domain/ReferencesJson.java Outdated
@@ -0,0 +8,4 @@
*/
@Data
public class ReferencesJson {
upagge commented 2023-01-20 18:46:13 +03:00
Owner
Copy Link

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

@Data
public class ReferencesJson {

    @JsonProperty("full")
    private String fullReference;
    
}
Код стайл в данном проекте это перенос строки в начале класса и в конце. Пример ``` @Data public class ReferencesJson { @JsonProperty("full") private String fullReference; } ```
DmitrySheyko commented 2023-01-22 00:16:21 +03:00
Author
Member
Copy Link

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

Стиль исправил
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
Fix: внесены правки по результатам ревью 1. 30ff49eb75
upagge worked for 1 week 2023-01-28 13:24:51 +03:00
DmitrySheyko added 1 commit 2023-02-03 20:22:57 +03:00
Feat: завершен основной функционал issue, выгрузка, удаление, уведомления 7e1c6c1f44
DmitrySheyko added 7 commits 2023-03-11 13:13:37 +03:00
Feat: создана сущность Issue и вспомогательные, приступаю к миграциям БД. b5ff193b49
Feat: создан changelog для таблиц сущности issue. 7396c2a2ef
Упростил сущность Issue 35783a55e7
Fix: внесены правки по результатам ревью 1. a514fcb74f
Feat: завершен основной функционал issue, выгрузка, удаление, уведомления 887f5a36f3
Feat: устранены конфликты с версией V.1.0.1 ee1bebb5e0
Merge remote-tracking branch 'origin/feature/issues_v1' into feature/issues_v1 e662a96eea 
# 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
upagge closed this pull request 2024-12-02 09:03:29 +03:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
upagge
Labels
Clear labels
bug
Баг в системе, который нужно исправить
feature
Фича, которую стоит реализовать
help wanted
Задача лежит в дальнем ящике
idea
Идеи, над которыми можно подумать
refactoring
No Label
feature
Milestone
No items
No Milestone Sprint 1
Projects
Clear projects
No project Версия бота 2.0
Assignees
Clear assignees
upagge
No Assignees DmitrySheyko
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Telegram-Bots/gitlab-notification#25
No description provided.
Delete Branch "feature/issues_v1"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?