-
Notifications
You must be signed in to change notification settings - Fork 6
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
Fixed #20 #33
Fixed #20 #33
Conversation
i havent run these changes on local but they look good to me, once i resolve the baseline migation issue on my local |
import { | ||
Body, | ||
Controller, | ||
Get, | ||
Post, | ||
Put, | ||
ValidationPipe, | ||
HttpStatus, | ||
Param, | ||
Req, | ||
Res, | ||
} from '@nestjs/common'; |
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.
Maybe we should add this: #34
|
||
async function bootstrap() { | ||
const app = await NestFactory.create(AppModule); | ||
const port = process.env.PORT || 3006; | ||
if (process.env.NODE_ENV !== 'production') { | ||
setupSwagger(app); |
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.
Very useful for development and coorditation between devs, thank you!
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.
This file is irrelavant (remove in the future)
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.
All changes look good to me! Thanks for your contribution. You could remove irrelevant DTO files in the future.
🎉 This issue has been resolved in version 1.3.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Dear Reviewer,
Please provide your .env file with the database url of your postgre instance.
Then:
prisma db push
to apply migrations to the dbyarn prisma:seed
to insert the test data to the dbnest start like
to start thelike
app, since it is the default one, simply runningnest start
will work as well.Please go to /docs to access the swagger ui.
Best,
Shpendi