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

remove 'false' option for servicediscovery, websocket, messageBroker and searc… #20664

Merged
merged 7 commits into from
Dec 28, 2022

Conversation

Tcharl
Copy link
Contributor

@Tcharl Tcharl commented Dec 25, 2022

remove somes deprecated jdl options
#20656

Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.
documentation update

@Tcharl Tcharl marked this pull request as ready for review December 26, 2022 11:47
@Tcharl Tcharl changed the title remove false for servicediscovery, websocket, messageBroker and searc… remove 'false' option for servicediscovery, websocket, messageBroker and searc… Dec 26, 2022
@Tcharl
Copy link
Contributor Author

Tcharl commented Dec 26, 2022

Looks like it also fixes a bug on searchEngine :-)

@Tcharl
Copy link
Contributor Author

Tcharl commented Dec 26, 2022

@DanielFran who to review this one?

app.jdl Outdated Show resolved Hide resolved
jdl/jhipster/default-application-options.ts Outdated Show resolved Hide resolved
jdl/jhipster/default-application-options.ts Outdated Show resolved Hide resolved
jdl/jhipster/monitoring-types.ts Outdated Show resolved Hide resolved
Co-authored-by: Daniel Franco <dandrfranco@gmail.com>
Copy link
Member

@mshima mshima left a comment

Choose a reason for hiding this comment

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

Exporting a wrapper object instead of using named exports is a bad practice it breaks esm lazy nature.

lib/index.mjs Outdated Show resolved Hide resolved
generators/base/generator-base.mjs Outdated Show resolved Hide resolved
@DanielFran
Copy link
Member

@Tcharl it will be needed to update documentation too, like in https://www.jhipster.tech/jdl/applications#options-in-applications

Tcharl and others added 2 commits December 28, 2022 11:42
Co-authored-by: Daniel Franco <dandrfranco@gmail.com>
Co-authored-by: Daniel Franco <dandrfranco@gmail.com>
@Tcharl
Copy link
Contributor Author

Tcharl commented Dec 28, 2022

@DanielFran documentation PR is done: jhipster/jhipster.github.io#1253

@Tcharl
Copy link
Contributor Author

Tcharl commented Dec 28, 2022

OK to merge guys? @DanielFran or @mshima ?

@DanielFran DanielFran merged commit cfbdbe3 into jhipster:main Dec 28, 2022
@DanielFran DanielFran added this to the 8.0.0-beta.1 milestone Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants