-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Entity blueprint: allow blueprints to create custom configs. #10163
Conversation
623e8ec
to
6f31c02
Compare
d702234
to
5d0db8b
Compare
5d0db8b
to
9b972fe
Compare
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
9b972fe
to
ad584aa
Compare
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 you resolve conflicts and handle my suggestions? Then we'll be ready to merge that.
Thanks!
e761d98
to
278f26c
Compare
@murdos done. Thanks. |
@mshima You need to lint fix the unit test source file, build is failing because of that: https://travis-ci.org/jhipster/generator-jhipster/jobs/587294729 |
Make tests cases more precise by testing keys and values. Co-Authored-By: Aurélien Mino <aurelien.mino@gmail.com> lint fix
278f26c
to
b724cef
Compare
@murdos thanks. |
Fixes #10162
Please make sure the below checklist is followed for Pull Requests.
Travis tests are green
Tests are added where necessary
Documentation is added/updated where necessary
Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed