-
-
Notifications
You must be signed in to change notification settings - Fork 113
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
looks like only the parse picks it up in the rnhipster call.. ? as it seems valid, in real-time in the JDL studio - prior to download #124
Comments
Please provide steps to reproduce this issue. |
I had to prefix the entity Account with Plant.. see pics.. JDL studio seemed to tell me it was ok with 'Account' as entity name.. then dloaded and moved jdl to backend and ran rnhipster jdl from the client referencing the jdl in the backend. sooooo rnhipster jdl ran and parsed and complained about the word Account of course jdl studio may intend for reasons to not check for reserved word usage,, thats fine.. all coolio. of course I have upper case in the App properties for 'React' and postgres... which the JDL studio didnt tell me the parser would complain about.. thats trivial.. quick edit.. but studio could check on those properties. |
Can you please include your JDL? |
below is the correction to pass the rnhipster jdl parser. the original simply had the word 'Account' not 'PlantAccount' and of course I had 'React' and 'Postgres' not full lower case.. and the JDL studio didnt flag the caps on those. still learning jdl of course. cheers. application { entity PlantAccount { entity Accounttype { entity Preferences { entity Datapermissions{ relationship OneToMany { } enum Language { dto PlantAccount, Accounttype, Preferences, Datapermissions with mapstruct |
arhhh.. the jdl should not include the application config IF im just importing new entities after a initial app gen.. this aspect is not truly specific to this issue though |
not sure if the JDL studio checks for reserved words prior to download and parse
The text was updated successfully, but these errors were encountered: