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

False value should be correctly handled for configs with choices #28194

Closed
2 tasks done
mshima opened this issue Dec 14, 2024 · 0 comments · Fixed by #28236
Closed
2 tasks done

False value should be correctly handled for configs with choices #28194

mshima opened this issue Dec 14, 2024 · 0 comments · Fixed by #28236

Comments

@mshima
Copy link
Member

mshima commented Dec 14, 2024

Overview of the issue

Properties like messageBroker that accepts kafka | pulsar | no values should handle correctly false values by falling back to no.

JHipster info

Welcome to JHipster v8.7.3

Welcome to the JHipster Information Sub-Generator

docker@0.0.1-SNAPSHOT /Users/mshima/git/jhipster-samples/docker
└── (empty)
JHipster configuration, a .yo-rc.json file generated in the root folder
.yo-rc.json file
{
  "generator-jhipster": {
    "baseName": "docker",
    "creationTimestamp": 1734180969839,
    "entities": [],
    "jhipsterVersion": "8.7.3",
    "languages": [
      "en"
    ],
    "messageBroker": false,
    "nativeLanguage": "en",
    "samplesFolder": "/Users/mshima/git/jhipster-samples"
  }
}
Environment and Tools

openjdk version "17.0.5" 2022-10-18
OpenJDK Runtime Environment Temurin-17.0.5+8 (build 17.0.5+8)
OpenJDK 64-Bit Server VM Temurin-17.0.5+8 (build 17.0.5+8, mixed mode)

git version 2.30.1 (Apple Git-130)

node: v18.20.5
npm: 10.9.2

Docker version 27.4.0, build bde2b89

JDL for the Entity configuration(s) entityName.json files generated in the .jhipster directory
JDL entity definitions

Congratulations, JHipster execution is complete!
If you find JHipster useful consider sponsoring the project https://www.jhipster.tech/sponsors/

Thanks for using JHipster!

  • jhipster info output is mandatory for bug reports. This will allow us to use automated tests and genarate the broken sample using jhipster from-issue command.
Motivation for or Use Case
Reproduce the error
Related issues

#28038 and similar issues.
#28193

Suggest a Fix
JHipster Version(s)
Browsers and Operating System
  • Tickets opened without reproduction steps or that doesn't follows the template recommendation will be closed.
  • [ x Checking this box is mandatory (this is just to show you read everything)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants