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

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

Open
johnda98 opened this issue Nov 8, 2021 · 5 comments

Comments

@johnda98
Copy link

johnda98 commented Nov 8, 2021

not sure if the JDL studio checks for reserved words prior to download and parse
Screen Shot 2021-11-08 at 9 25 46 AM

@johnda98 johnda98 changed the title looks like only the parse picks it up.. is there a parse checker in the studio ? as it seems valid real-time in the JDL studio. 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 Nov 8, 2021
@mraible
Copy link
Contributor

mraible commented Nov 8, 2021

Please provide steps to reproduce this issue.

@johnda98
Copy link
Author

johnda98 commented Nov 8, 2021

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.

just a fyi.. no biggy
Screen Shot 2021-11-08 at 10 21 11 AM

Screen Shot 2021-11-08 at 10 31 15 AM

@mraible
Copy link
Contributor

mraible commented Nov 8, 2021

Can you please include your JDL?

@johnda98
Copy link
Author

johnda98 commented Nov 8, 2021

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 {
config {
baseName Plantbackend,
applicationType monolith,
packageName com.plantidote.plantapi,
authenticationType jwt,
prodDatabaseType postgresql,
clientFramework react
}
entities *
}

entity PlantAccount {
accountid String required
profilesmartcontractid String required
}

entity Accounttype {
accounttype String required
}

entity Preferences {
preferenceid String required
}

entity Datapermissions{
coretoplatform Boolean required
noncoretoplatform Boolean required
opennessmeasure Integer required
}

relationship OneToMany {
Accounttype{Account(accountid)} to PlantAccount{Accounttype(accounttype)}
PlantAccount{Preferences(preferenceid)} to Preferences{Account(accountid)}
PlantAccount to Datapermissions

}

enum Language {
ENGLISH,
SPANISH
}

dto PlantAccount, Accounttype, Preferences, Datapermissions with mapstruct
service PlantAccount, Accounttype, Preferences, Datapermissions with serviceClass
paginate PlantAccount with infinite-scroll
paginate PlantAccount with pagination

@johnda98
Copy link
Author

johnda98 commented Nov 9, 2021

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

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