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

Add id dropdown to one-to-one-relationship #1757

Merged

Conversation

andidev
Copy link
Member

@andidev andidev commented Jul 22, 2015

  • ask for other entity relationship name on owner side
  • add underscore javascript library to generator
  • sort packages in package.js in alphabetic order
  • expose a new util object in templates (added contains method)
  • add dropdown to select id on relationship owner entity in create/update dialog
  • add presentation of relatinship id in all ohter html crud pages (on owner side)
  • add filter parameter to findAll in REST query which returns only objects with no relationship
  • query with filter parameter when getting the ids for one-to-one relationships in entity-dialog-controller
    fix There is no foreign key inserted for one-to-one relationship #1468

@jdubois
Copy link
Member

jdubois commented Jul 25, 2015

Sorry I did some merging and broke your PR.
I have never been a big fan of one-to-one relationships, but yes this would be an improvement

- ask for other entity relationship name on owner side
- add underscore javascript library to generator
- sort packages in package.js in alphabetic order
- expose a new util object in templates (added contains method)
- add dropdown to select id on relationship owner entity in create/update dialog
- add presentation of relatinship id in all ohter html crud pages (on owner side)
- add filter parameter to findAll in REST query which returns only objects with no relationship
- query with filter parameter when getting the ids for one-to-one relationships in entity-dialog-controller
fix jhipster#1468
@andidev andidev force-pushed the add-id-dropdown-to-one-to-one-relationships branch from a493e09 to 5581ab7 Compare July 26, 2015 12:05
@andidev
Copy link
Member Author

andidev commented Jul 26, 2015

Don't merge this yet. This will fail with java7 option since I used some java 8 streaming. Need to fix that first.

@andidev
Copy link
Member Author

andidev commented Jul 27, 2015

This should okay to merge now

jdubois added a commit that referenced this pull request Jul 27, 2015
…elationships

Add id dropdown to one-to-one-relationship
@jdubois jdubois merged commit 179d7e0 into jhipster:master Jul 27, 2015
@jdubois jdubois modified the milestone: Untracked PRs Oct 21, 2015
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

Successfully merging this pull request may close these issues.

There is no foreign key inserted for one-to-one relationship
2 participants