feature/issues_v1 #25
No reviewers
Labels
No Label
bug
feature
help wanted
idea
refactoring
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Telegram-Bots/gitlab-notification#25
Loading…
Reference in New Issue
Block a user
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?
Dmitry Sheyko.
Pull Request 1:
В целом все хорошо. Однако, нам для работы уведомлений не обязательно сохранять всю информацию по Issue, которая поступает. Стоит ограничиться только набором необходимых для работы полей. Я удалил лишние классы, а также лишние поля в Issue. Теперь нужно привести схему к новой доменной модели. Исправлять Json классы не нужно, там все хорошо.
@ -0,0 +1,6 @@
package dev.struchkov.bot.gitlab.context.domain;
public enum IssueType {
ISSUE,
Нужно писать в одном стиле. Сейчас даже в рамках одного ПР стиль отличается. За образец предлагаю взять MergeRequestState
Правки внес.
@ -0,0 +34,4 @@
*
* @author Dmitry Sheyko [17.01.2023]
*/
Лишний перенос строки, и комментарий непонятно что означает, почему это стоит учитывать и почему этот комментарий тут? Пагинация передается и настраивается. Комментарий стоит удалить.
Правки внес.
Комментарий думал оставить временно, чтобы не забыть осоенность которую нашел в документации. Привычка везде оставлять напоминания.
@ -0,0 +4,4 @@
import java.util.Set;
public class IssueRequestParser {
Пр должен иметь логическое заключение, это явно лишний клас. По хорошему, нужно удалить.
Удалил.
Логику понял. Внчачале работы не знал с чего начать и создал класс:)
@ -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">
Нужно актуализировать схему согласно упращенной доменной модели
ChangeSet доработал
@ -0,0 +44,4 @@
</createTable>
</changeSet>
<changeSet id="create-table-issue" author="Dmitry Sheyko">
пропущена дата в id
Даты внес
@ -0,0 +179,4 @@
</changeSet>
<changeSet id="create-table-issue_labels" author="Dmitry Sheyko">
<createTable tableName="issue_labels">
В целях следования единому стилю, нужно переименовать на issue_label
Переименовал
@ -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)">
labels -> label
Сделано
@ -0,0 +15,4 @@
*/
@Data
public class IssueJson {
Вот этот класс не обязательно упращать. Можно и даже нужно оставить его в таком виде.
Понятно
@ -0,0 +74,4 @@
private Integer discussionLocked;
@JsonProperty("issue_type")
private String issueType; //TODO выяснить зачем дублирует поле type
TODO можно удалить, скоре всего это просто оставлено для обратной совместимости со старым апи
Удалено
@ -0,0 +5,4 @@
/**
* @author Dmitry Sheyko [17.01.2023]
*/
Лишние переносы нужно удалить во всех классах
Переносы удалил
@ -0,0 +8,4 @@
*/
@Data
public class ReferencesJson {
Код стайл в данном проекте это перенос строки в начале класса и в конце.
Пример
Стиль исправил
Checkout
From your project repository, check out a new branch and test the changes.