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

Update relationships.md #1338

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Lalithkumarponnambalam
Copy link

Update relationships.md content

The following issue will be rectified:

I have referenced the "joint fields" from the JHipster Pro documentation. The correct link for the documentation is:

@CLAassistant
Copy link

CLAassistant commented Mar 21, 2024

CLA assistant check
All committers have signed the CLA.

Copy link

netlify bot commented Mar 21, 2024

Deploy Preview for jhipster-site ready!

Name Link
🔨 Latest commit 04ce7e9
🔍 Latest deploy log https://app.netlify.com/sites/jhipster-site/deploys/65fc01e89e1e600008a135a3
😎 Deploy Preview https://deploy-preview-1338--jhipster-site.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Lalithkumarponnambalam Lalithkumarponnambalam marked this pull request as draft March 21, 2024 10:18
@Lalithkumarponnambalam Lalithkumarponnambalam marked this pull request as ready for review March 21, 2024 10:18
@OmarHawk
Copy link
Contributor

Is this correct? To me the described syntax rather means "show name in any referencing attribute, like dropdown or list view link", the join will still always be executed based on the foreign key relationship afaik.

@Lalithkumarponnambalam
Copy link
Author

@OmarHawk yes this is correct. In the JHipster Pro Documentation, there is an explanation of join fields in relationships, which can be found at the following link: JHipster Pro Documentation - Relationships with Joint Fields.

@OmarHawk
Copy link
Contributor

OmarHawk commented Apr 3, 2024

Still not convinced. We are using the syntax and it works as I did describe. See also the syntax description here: https://www.jhipster.tech/jdl/relationships#syntax ---> <display field>

Also the .pro site looks kind of unofficial and rather outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants