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

use defaults utility at relationships. #23931

Merged
merged 3 commits into from
Oct 21, 2023
Merged

Conversation

mshima
Copy link
Member

@mshima mshima commented Oct 20, 2023

Related to #23917.


Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

@mshima
Copy link
Member Author

mshima commented Oct 20, 2023

This doesn’t fix the problem, but the api for loading relationships is there.
Fetching not ownerSide is only useful for view, unless we are going to cascade, so we probably need to add a flag for write/read.

@hide212131
Copy link
Contributor

I've reviewed the generated code and have a general understanding of the concept.

@mshima mshima changed the title Eager load relationships use defaults utility at relationships. Oct 20, 2023
@mshima mshima marked this pull request as ready for review October 20, 2023 21:19
@DanielFran DanielFran merged commit 8b296ba into jhipster:main Oct 21, 2023
48 checks passed
@deepu105 deepu105 added this to the 8.0.0 milestone Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants