Skip to content

Added Entities#7

Open
prathambatra wants to merge 3 commits into
coding-blocks:masterfrom
prathambatra:create-entities
Open

Added Entities#7
prathambatra wants to merge 3 commits into
coding-blocks:masterfrom
prathambatra:create-entities

Conversation

@prathambatra

Copy link
Copy Markdown
Contributor

Signed-off-by: prathambatra batrapratham999@gmail.com
fixed #4

Signed-off-by: prathambatra <batrapratham999@gmail.com>
@boss-contributions-bot

Copy link
Copy Markdown

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.

@prathambatra

Copy link
Copy Markdown
Contributor Author

@championswimmer unit test is falling because enum is not supported type in sqlite. I think createTestTypeORMConfig should also have type postgres.

@prathambatra

Copy link
Copy Markdown
Contributor Author

@championswimmer

@championswimmer

Copy link
Copy Markdown
Contributor

Please use simple-enum type it will work for both

typeorm/typeorm#3700

@championswimmer

Copy link
Copy Markdown
Contributor

also ShieldAgent.managerId should be proxy for the user.managerID

same for Superhero.handlerId

@championswimmer

Copy link
Copy Markdown
Contributor

also you can't have @joincolumn on both sides.

@prathambatra

Copy link
Copy Markdown
Contributor Author

also you can't have @joincolumn on both sides.

I haven't made bidirectional relations.

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.

create user, superhero, agent entities with relations

2 participants