Skip to content
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

Add api #1

Merged
merged 11 commits into from
Jun 7, 2024
Merged

Add api #1

merged 11 commits into from
Jun 7, 2024

Conversation

asavershin
Copy link
Owner

No description provided.

@asavershin asavershin added the enhancement New feature or request label Apr 2, 2024
@asavershin asavershin self-assigned this Apr 2, 2024
@asavershin
Copy link
Owner Author

asavershin commented Apr 4, 2024

PostgreSQL выбрана для хранения метаинформации, так как +- известна структура этой самой метаинформации. К тому же хранение пользователей в SQL бд, а картинки, например, в Mongo создаст дополнительную кодовую нагрузку на уровне application по контролю консистентности данных

@MaximKozlyuk
Copy link

Работа хорошая
Ты видимо эту структуру папок с in/out взял из гексоганальной архитектуры? Ну интересно чем конкретно вдохновлялся.

Сложно это критиковать если честно, что-то аналогичное сейчас в команде в свежих микросервисах используем. Мы даже не стали пока выделять отдельные аннотации типа твоих Command хотя возможно стоит:)
Нравится что логика по небольшим классам упакована под четким интерфейсом.

@asavershin
Copy link
Owner Author

asavershin commented May 25, 2024

По большей части вдохновлялся из Эванса DDD и Вона Вернона. Иерархию уже сам накидывал примерно рассуждая, как бы это лучше было с точки зрения того же DDD, по мелочёвки ещё смотрел видосы ребят на шарпах. Но тут всё очень индивидуально, конкретно я так захотел сделать. А так, да, в основном глаз пал на гексоганалку в качестве обёртки для DDD.

@asavershin
Copy link
Owner Author

Overall Project 81.5% -18.5% 🍏
Files changed 81.5% 🍏

File Coverage
AuthenticatedUserRepositoryImpl.java 100% 🍏
UserRepositoryImpl.java 100% 🍏
ImageRepositoryImpl.java 100% 🍏
PartOfResources.java 100% 🍏
ResourceOwnershipException.java 100% 🍏
IsEntityFound.java 100% 🍏
DeleteImageOfUserImpl.java 100% 🍏
GetImageOfUserImpl.java 100% 🍏
GetPartImagesOfUserImpl.java 100% 🍏
StoreImageOfUserImpl.java 100% 🍏
FullName.java 100% 🍏
Credentials.java 100% 🍏
UserId.java 100% 🍏
RegisterNewUserImpl.java 100% 🍏
JwtServiceIml.java 100% 🍏
GetNewCredentialsUsingRefreshTokenImpl.java 100% 🍏
GetNewCredentialsImpl.java 100% 🍏
ApplicationCredentials.java 100% 🍏
TokenRepositoryIml.java 100% 🍏
CacheRepositoryIml.java 100% 🍏
ImageId.java 100% 🍏
ImageExtension.java 100% 🍏
MetaData.java 100% 🍏
ImageServiceImpl.java 100% 🍏
TryToLoginImpl.java 86.84% -13.16% 🍏
MinioServiceIml.java 73.27% -26.73% 🍏
ImageNameWithExtension.java 71.08% -28.92% 🍏
Image.java 62.63% -37.37% 🍏
User.java 58.66% -41.34% 🍏
GetPartOfImagesForAuthenticatedUserImpl.java 50% -50% 🍏
AuthenticatedUser.java 28.91% -71.09%
AuthException.java 0%
FileException.java 0%

@asavershin asavershin merged commit c3dc850 into master Jun 7, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants