Skip to content

Fix models #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jul 25, 2024
Merged

Fix models #6

merged 4 commits into from
Jul 25, 2024

Conversation

funnyDevGirl
Copy link

No description provided.

@s-chepurnov
Copy link

Студенты делают 4 проект и почти всегда указывают дату создания объекта в конструкторе класса доменной модели, но это не верно.
Дата создания объекта это доп.техническая информация и она не должна требоваться в конструкторе, но должна устанавливаться на уровне репозитория через сетер.
Этот код даёт корректный пример установки даты создания объекта.
Также необходимо использовать классы Instant или LocalDateTime, но студенты ошибочно используют Timestamp.

@HelenOne HelenOne requested a review from Malcom1986 July 11, 2024 07:57
@Malcom1986
Copy link
Contributor

А давайте тут LocalDateTime будем использовать, мы везде его используем в курсах

@Malcom1986
Copy link
Contributor

@funnyDevGirl Алина, добрый день! Тесты упали, посмотрите пожалуйста https://github.com/hexlet-components/java-javalin-example/actions/runs/10029297135/job/27731026455?pr=6#step:6:207

Там еще схему таблиц нужно будет поменять

@funnyDevGirl
Copy link
Author

Malcom1986 Максим, добрый день! Схему поменяла. Прошу посмотреть. Спасибо.

@Malcom1986 Malcom1986 merged commit 2c0aa28 into hexlet-components:main Jul 25, 2024
2 checks passed
@Malcom1986
Copy link
Contributor

Принял, спасибо!

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.

3 participants