-
Notifications
You must be signed in to change notification settings - Fork 27
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
(feat) Add support for generating modules #73
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
plopfile.mjs
Outdated
{ | ||
type: "input", | ||
name: "name", | ||
message: "Name your resource: ", |
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.
How about we use module instead of resource?
app.use(bodyParser.urlencoded({ extended: true })); | ||
app.use(bodyParser.json()); | ||
|
||
app.use('/organization account', OrganizationAccountRouter.getRoutes()); |
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.
can this be asked as a question too - like what rest base url you want for the module? If user presses Enter you can fall back to default one.
Also we are missing a hyphen
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.
exciting!
I couldn't see the app.ts changes for the module. If we intend not to do that can we show a message to the user of what is required from his side once the generation is done?
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Deployment (boilerplate-mern) is available at - https://4a492aace6a73cd.preview.platform.jalantechnologies.com |
Minimum allowed line rate is |
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.
lgtm
This PR is being closed due to inactivity. Our team has logged the issue and this will get taken care of in future releases. |
Description
generates module on running
npm run plop
Database schema changes
NA
Tests
Automated test cases added
Manually tested
Kindly review
organization-account
generated using plop and template