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

Add an import-jdl command #29

Closed
mraible opened this issue Feb 24, 2018 · 5 comments
Closed

Add an import-jdl command #29

mraible opened this issue Feb 24, 2018 · 5 comments

Comments

@mraible
Copy link
Collaborator

mraible commented Feb 24, 2018

jhipster/generator-jhipster-react-native#31

@MInesGomes
Copy link

Hi @mraible,

Is it difficult to add option -yesToAll and the path for backend
That would reply yes to all and considered my Backend Path?
Something like?
yo jhipster-ionic:entity MyEntity -yesToAll ../MyBackEnd

With 100 entities is just run that and we have the same as jdl :-)

Example: When I generate backend I get only for the changed entities:

Entity NmsIntegratorServer generated successfully.
Entity NmsIntServerBrand generated successfully.
Entity NmsChgNetwork generated successfully.
Entity NmsAlert generated successfully.
Entity NmsPatientState generated successfully.
Entity NmsAlertReason generated successfully.
Entity NmsAlertLog generated successfully.
Entity NmsAlertPrecedence generated successfully.
Entity NmsChatRead generated successfully.
Entity NmsChatReceived generated successfully.
Entity NmsFamily generated successfully.

I would replace by

yo jhipster-ionic:entity NmsIntegratorServer -yesToAll ../MyBackEnd
yo jhipster-ionic:entity NmsIntServerBrand -yesToAll ../MyBackEnd
yo jhipster-ionic:entity NmsChgNetwork -yesToAll ../MyBackEnd
yo jhipster-ionic:entity NmsAlert -yesToAll ../MyBackEnd
yo jhipster-ionic:entity NmsPatientState -yesToAll ../MyBackEnd
yo jhipster-ionic:entity NmsAlertReason -yesToAll ../MyBackEnd
yo jhipster-ionic:entity NmsAlertLog -yesToAll ../MyBackEnd
yo jhipster-ionic:entity NmsAlertPrecedence -yesToAll ../MyBackEnd
yo jhipster-ionic:entity NmsChatRead -yesToAll ../MyBackEnd
yo jhipster-ionic:entity NmsChatReceived -yesToAll ../MyBackEnd
yo jhipster-ionic:entity NmsFamily -yesToAll ../MyBackEnd

And run it without having to answer nothing it would be almost the same as having jdl :-)

What do you think?

@mraible
Copy link
Collaborator Author

mraible commented Jul 30, 2019

@Entrecampos I originally tried to do yo jhipster-ionic:entity *, but was unable to capture the asterisk character. It seems it's being swallowed by Yeoman.

I agree that we should have one command that generates all entities. I think import-jdl is the most familiar to people using JHipster, so that's what I'd like to support.

@MInesGomes
Copy link

@mraible Yes it makes sense jdl. But if possible -yesToAll would be great :-)

@mraible
Copy link
Collaborator Author

mraible commented Aug 24, 2019

@deepu105 I see that import-jdl.js in JHipster is not a generator, so it's not as easy to extend as the entity generator was. Do you know if it's possible to extend import-jdl from JHipster and implement it in Ionic (which is a Yeoman module, not a blueprint). If not, I can probably just copy/paste the whole thing.

Thanks!

Matt

@mraible
Copy link
Collaborator Author

mraible commented Sep 12, 2019

Fixed by #159.

@mraible mraible closed this as completed Sep 12, 2019
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

No branches or pull requests

2 participants