Skip to content

feature/BJS2-96431#3514

Open
flesheatingcat wants to merge 7 commits intophoenix-master-stream12from
feature/BJS2-96431
Open

feature/BJS2-96431#3514
flesheatingcat wants to merge 7 commits intophoenix-master-stream12from
feature/BJS2-96431

Conversation

@flesheatingcat
Copy link
Copy Markdown

Recommendation Request Service

@github-actions
Copy link
Copy Markdown

⚠️ ОШЫБКА: Сборка завалилась: либо ошибки компиляции, либо не прошли тесты, либо возникли ошибки в стиле кода. Пожалуйста, проверь логи и внеси соответствующие изменения в ПР. После этого красная ошибка ниже должна пропасть, но этот комментарий останется - так и должно быть.

@github-actions
Copy link
Copy Markdown

⚠️ ОШЫБКА: Сборка завалилась: либо ошибки компиляции, либо не прошли тесты, либо возникли ошибки в стиле кода. Пожалуйста, проверь логи и внеси соответствующие изменения в ПР. После этого красная ошибка ниже должна пропасть, но этот комментарий останется - так и должно быть.

Copy link
Copy Markdown

@SophieChe SophieChe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Виктор, оставила комменты

String messageContains,
RequestStatus status
) {
@AssertTrue(message = "Either Requester or Received ID must be provided.")
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Мне кажется, этот метод можно заменить на аннотации библиотеки jakarta.validation. Посмотри материал про нее, мне кажется, ты ее нигде не используешь никогда.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Валидацию использую, где надо. Здесь же мне нужно, чтоб одно из полей присутствовало, не оба. Как такое сделать через jakarta я не нашёл, поэтому запилил так.

@github-actions
Copy link
Copy Markdown

⚠️ ОШЫБКА: Сборка завалилась: либо ошибки компиляции, либо не прошли тесты, либо возникли ошибки в стиле кода. Пожалуйста, проверь логи и внеси соответствующие изменения в ПР. После этого красная ошибка ниже должна пропасть, но этот комментарий останется - так и должно быть.

@github-actions
Copy link
Copy Markdown

⚠️ ОШЫБКА: Сборка завалилась: либо ошибки компиляции, либо не прошли тесты, либо возникли ошибки в стиле кода. Пожалуйста, проверь логи и внеси соответствующие изменения в ПР. После этого красная ошибка ниже должна пропасть, но этот комментарий останется - так и должно быть.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants