-
-
Notifications
You must be signed in to change notification settings - Fork 92
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
added some mappers #295
added some mappers #295
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for improving the test coverage and for making the code less cumbersome by adding the generated converters ❤️
I would like you to take a look at the failing test - this is the only serious blocker. If you have resources, take a look at other comments as well, but remember that "nitpicking" notes are optional 👍
src/main/java/com/zufar/icedlatte/favorite/converter/FavoriteListDtoConverter.java
Outdated
Show resolved
Hide resolved
src/main/java/com/zufar/icedlatte/favorite/converter/FavoriteListDtoConverter.java
Outdated
Show resolved
Hide resolved
src/main/java/com/zufar/icedlatte/favorite/converter/FavoriteListDtoConverter.java
Outdated
Show resolved
Hide resolved
src/main/java/com/zufar/icedlatte/favorite/converter/FavoriteListDtoConverter.java
Outdated
Show resolved
Hide resolved
src/main/java/com/zufar/icedlatte/favorite/converter/FavoriteListDtoConverter.java
Outdated
Show resolved
Hide resolved
src/test/java/com/zufar/icedlatte/favorite/converter/FavoriteItemDtoConverterTest.java
Show resolved
Hide resolved
src/test/java/com/zufar/icedlatte/favorite/converter/FavoriteListDtoConverterTest.java
Outdated
Show resolved
Hide resolved
src/test/java/com/zufar/icedlatte/favorite/converter/FavoriteListDtoConverterTest.java
Outdated
Show resolved
Hide resolved
src/test/java/com/zufar/icedlatte/review/api/ProductAverageRatingProviderTest.java
Outdated
Show resolved
Hide resolved
...test/java/com/zufar/icedlatte/favorite/converter/ListOfFavoriteProductsDtoConverterTest.java
Show resolved
Hide resolved
src/test/java/com/zufar/icedlatte/favorite/converter/FavoriteListDtoConverterTest.java
Outdated
Show resolved
Hide resolved
src/test/java/com/zufar/icedlatte/review/api/ProductAverageRatingProviderTest.java
Outdated
Show resolved
Hide resolved
src/test/java/com/zufar/icedlatte/favorite/converter/FavoriteListDtoConverterTest.java
Outdated
Show resolved
Hide resolved
src/test/java/com/zufar/icedlatte/favorite/converter/FavoriteItemDtoConverterTest.java
Show resolved
Hide resolved
…-mapstruct' into feature/IL-287-custom-mappers-to-mapstruct # Conflicts: # src/main/java/com/zufar/icedlatte/review/api/ProductReviewsStatisticsProvider.java
…to-mapstruct-clone COPY of #295: added some mappers by Victoria Ziborova
No description provided.