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

feature/issues_v1 #25

Open
DmitrySheyko wants to merge 12 commits from feature/issues_v1 into develop
pull from: feature/issues_v1
merge into: Telegram-Bots:develop
Conversation 0 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
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
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
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
View command line instructions.

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.
Reviewers
No reviewers
upagge
Labels
Clear labels   
bug

Баг в системе, который нужно исправить

  
feature

Фича, которую стоит реализовать

  
help wanted

Задача лежит в дальнем ящике

  
idea

Идеи, над которыми можно подумать

  
refactoring
No Label
bug
feature
help wanted
idea
refactoring
Milestone
Clear milestone
No items
No Milestone
Sprint 1
Projects
Clear projects
No project
Версия бота 2.0
Assignees
Clear assignees
No Assignees
DmitrySheyko
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.
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?