-
Notifications
You must be signed in to change notification settings - Fork 8
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 Entities #7
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: prathambatra <[email protected]>
Thanks @prathambatra, for opening the pull request! 🙌 One of our mentors will review the pull request soon. ✅ Star ⭐ this project and tweet 🐦 about your contributions. |
@championswimmer unit test is falling because enum is not supported type in sqlite. I think createTestTypeORMConfig should also have type postgres. |
Please use |
also ShieldAgent.managerId should be proxy for the user.managerID same for Superhero.handlerId |
also you can't have @joincolumn on both sides. |
I haven't made bidirectional relations. |
Signed-off-by: prathambatra [email protected]
fixed #4