Skip to content
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

Closed
wants to merge 3 commits into from

Conversation

Patel-world
Copy link
Contributor

Description

generates module on running npm run plop

image

Database schema changes

NA

Tests

Automated test cases added

  • NA

Manually tested

Kindly review organization-account generated using plop and template

@Patel-world Patel-world self-assigned this Aug 24, 2023
@Patel-world Patel-world linked an issue Aug 24, 2023 that may be closed by this pull request
@jalan-technologies-sonarqube

This comment has been minimized.

@Patel-world Patel-world requested a review from bbbneo333 August 24, 2023 12:14
@github-actions

This comment has been minimized.

@Patel-world Patel-world changed the title feat-add-support-for-generating-modules (feat) add support for generating modules Aug 24, 2023
@Patel-world Patel-world changed the title (feat) add support for generating modules (feat) Add support for generating modules Aug 24, 2023
@jalan-technologies-sonarqube

This comment has been minimized.

@github-actions

This comment has been minimized.

plopfile.mjs Outdated
{
type: "input",
name: "name",
message: "Name your resource: ",

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());

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

Copy link

@vjnvisakh-jtc vjnvisakh-jtc left a 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?

@jalan-technologies-sonarqube

This comment has been minimized.

@github-actions

This comment has been minimized.

@jalan-technologies-sonarqube
Copy link

Passed

Analysis Details

0 Issues

  • Bug0 Bugs
  • Vulnerability0 Vulnerabilities
  • Code Smell0 Code Smells

Coverage and Duplications

  • No coverage informationNo coverage information (0.00% Estimated after merge)
  • 3 percent duplication0.00% Duplicated Code (0.00% Estimated after merge)

Project ID: jalantechnologies_boilerplate-mern

View in SonarQube

@github-actions
Copy link

github-actions bot commented Sep 6, 2023

Deployment (boilerplate-mern) is available at - https://4a492aace6a73cd.preview.platform.jalantechnologies.com

@github-actions
Copy link

github-actions bot commented Sep 6, 2023

Code Coverage

Package Line Rate Branch Rate Health
backend 96% 50%
backend.modules.access-token 100% 100%
backend.modules.access-token.internal 100% 100%
backend.modules.access-token.rest-api 98% 100%
backend.modules.account 77% 100%
backend.modules.account.internal 94% 67%
backend.modules.account.internal.store 92% 50%
backend.modules.account.rest-api 100% 100%
backend.modules.communication 83% 75%
backend.modules.communication.internals 90% 56%
backend.modules.config 92% 88%
backend.modules.error 100% 100%
backend.modules.logger 78% 100%
backend.modules.logger.internals 69% 45%
backend.modules.organization-account 100% 100%
backend.modules.organization-account.internal.store 92% 50%
backend.modules.organization-account.rest-api 87% 100%
backend.modules.task 90% 100%
backend.modules.task.internal 95% 60%
backend.modules.task.internal.store 92% 50%
backend.modules.task.rest-api 94% 100%
Summary 89% (583 / 655) 67% (60 / 90)

Minimum allowed line rate is 60%

Copy link

@vjnvisakh-jtc vjnvisakh-jtc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@bbbneo333
Copy link
Contributor

bbbneo333 commented Dec 15, 2023

This PR is being closed due to inactivity. Our team has logged the issue and this will get taken care of in future releases.

@bbbneo333 bbbneo333 closed this Dec 15, 2023
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.

(feat) Add support for generating modules
4 participants