search

jhipster / jdl-studio

JDL Studio is an online JHipster Domain Language visual editor
https://start.jhipster.tech/jdl-studio/
Apache License 2.0
405 stars 114 forks source link

Keyword 'unique' not supported #60

Closed murdos closed 4 years ago

murdos commented 5 years ago

Not sure if this issue belongs to jhipster-online or here...

If you use the following JDL in https://start.jhipster.tech/jdl-studio/, the chart doesn't update anymore, until you remove the unique keyword:

entity A {
    code String unique
}
deepu105 commented 5 years ago

Didn't we remove support for unique keyword? Does the generator support it?

On Wed, 5 Dec 2018, 5:18 pm Aurélien Mino <notifications@github.com wrote:

Not sure if this issue belongs to jhipster-online https://github.com/jhipster/jhipster-online or here...

If you use the following JDL in https://start.jhipster.tech/jdl-studio/, the chart doesn't update anymore, until you remove the unique keyword:

entity A {

code String unique

}

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jhipster/jdl-studio/issues/60, or mute the thread https://github.com/notifications/unsubscribe-auth/ABDlF9SavjfNgK4V9WCzP42ZtTpLU3wxks5u1_HpgaJpZM4ZDEbr .

murdos commented 5 years ago

Yes, it supports it. Why support for it would be removed?

deepu105 commented 5 years ago

I need to check since I remember there were some issues.

Thanks & Regards, Deepu

On Wed, Dec 12, 2018 at 4:37 PM Aurélien Mino notifications@github.com wrote:

Yes, it supports it. Why support for it would be removed?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/jhipster/jdl-studio/issues/60#issuecomment-446631281, or mute the thread https://github.com/notifications/unsubscribe-auth/ABDlF5rkhB5eVyikTHwDay7wQfxi6aemks5u4SK2gaJpZM4ZDEbr .

ernsyn commented 5 years ago

Hi there,

Any updates on this issue? I had the same problem and it took me a while to realise it's due to the unique keyword.

clakech commented 5 years ago

Every JHipsters users with "unique" in their JDL cannot use the JDL-studio anymore. I have errors in the browser console when I use "unique" constraint

Would this change avoid this error https://github.com/jhipster/jdl-studio/pull/64 ?

deepu105 commented 5 years ago

@MathieuAA could you take a look if you have time, i'm having difficulty finding time here :cry:

jdubois commented 5 years ago

@deepu105 for me this the fix is merged, what else is there to do? Of course I need to do a new release to get in on start.jhipster.tech

deepu105 commented 5 years ago

I believe the fix only adds syntax highlight. In order for it to work the core version needs to be updated and this needs to be released to jh online. I dont know if the last part was already done. In that case its fine

MathieuAA commented 5 years ago

@deepu105 so do I unfortunately.

deepu105 commented 5 years ago

May be this is quite old and not an issue anymore :) anyway will check when I find time

Thanks & Regards, Deepu

On Mon, Apr 8, 2019 at 8:54 AM Mathieu ABOU-AICHI notifications@github.com wrote:

@deepu105 https://github.com/deepu105 so do I unfortunately.

I have trouble understanding why this is an issue coming from this project...

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jhipster/jdl-studio/issues/60#issuecomment-480706972, or mute the thread https://github.com/notifications/unsubscribe-auth/ABDlF0X2thIfJwEuseBHjKJBxt6PWJElks5veueTgaJpZM4ZDEbr .