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

Relationship with Authority not possible #22106

Closed
1 task done
OmarHawk opened this issue May 12, 2023 · 1 comment · Fixed by #24965
Closed
1 task done

Relationship with Authority not possible #22106

OmarHawk opened this issue May 12, 2023 · 1 comment · Fixed by #24965

Comments

@OmarHawk
Copy link
Contributor

Overview of the issue

When using Authority as part of a relationship, the application build is failing. See linked Pull Request, where I added this kind of thing into one of the tests.

Motivation for or Use Case

We have a use case where we want to use the existing Authority entity as part of a ManyToMany relationship. I tried it with current main branch, but that produces an error, even when using the with builtInEntity syntax.

My workaround was now building the Aplication with a "FakeAuthority" entity that is the very same as normal Authority (one single column with name String which is also the Id), and then switching all generated references to Authority and getting rid of any bi-directional thing. Will serve as a basis for adjusting the generator to make it work out-of-the-box (see below)

Reproduce the error

#22105

Related issues

#14873 / #14596

Suggest a Fix

I'll try to enhance the Draft Pull request with a fix for the failing tests to generate the application similar to what i achieved manually in there in the next week unless someone points me to something that I did wrong or someone else working on that beforehand. :)

JHipster Version(s)

main branch (and also 7.9.3 where stuff like that isn't supported at all)

JHipster configuration
Entity configuration(s) entityName.json files generated in the .jhipster directory
Browsers and Operating System
  • Checking this box is mandatory (this is just to show you read everything)
Copy link
Contributor

This issue is stale because it has been open for too long without any activity.
Due to the moving nature of jhipster generated application, bugs can become invalid.
If this issue still applies please comment otherwise it will be closed in 7 days

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.

3 participants