search

trebol-ecommerce / trebol-backend-monolith

Monolithic eCommerce backend web application that exposes a RESTful API.
MIT License
16 stars 21 forks source link

issue-56 remove @Basic, @NamedQuery, fetchType.Eager, modify serialVe… #59

Closed trangntt-016 closed 3 years ago

trangntt-016 commented 3 years ago

Hi @bglamadrid ,

Here are some changes I've just made in your entities:

bglamadrid commented 3 years ago

@trangntt-016 your changes looks great! I've reviewed and approved them. Thank you very much! Remember to link the issues you're targetting (if any) when you create another pull request 😉

trangntt-016 commented 3 years ago

Thanks Ben,

Sorry, I will link the issues in my next pull request 😁

On Sat, Oct 2, 2021 at 11:11 AM bglamadrid @.***> wrote:

@trangntt-016 https://github.com/trangntt-016 your changes looks great! I've reviewed and approved them. Thank you very much! Remember to link the issues you're targetting (if any) when you create another pull request 😉

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/trebol-ecommerce/spring-boot-backend/pull/59#issuecomment-932768138, or unsubscribe https://github.com/notifications/unsubscribe-auth/AON7H7E2WS5WW2GHO6PH4PDUE4OLHANCNFSM5FFU4AKA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.