Skip to content

Add HasOne Association - #7

Open
simozhan wants to merge 2 commits into
add_association_has_manyfrom
add_association_has_one
Open

Add HasOne Association#7
simozhan wants to merge 2 commits into
add_association_has_manyfrom
add_association_has_one

Conversation

@simozhan

Copy link
Copy Markdown
Owner

According to some PDC server's primary_key is multiple,
add a mapping in identity. Not sure if this a good implementation
or not.

JIRA: PDC-1685

1. Implement simple has_many association which can help to cache result
and add some test cases.
As pdc gem only reads data from pdc server, this patch only implement
get related association behavior, not including post action.
2. Change some rubocop configurations

Known issues:
1. Need more test cases.
2. Haven't change existing api which can use has_many association
like Release, will change it in future patches.

JIRA: PDC-1685
According to some PDC server's primary_key is multiple,
add a mapping in identity. Not sure if this a good implementation
or not.

JIRA: PDC-1685
@simozhan
simozhan force-pushed the add_association_has_one branch from aaf0847 to d67c65f Compare January 12, 2018 02:21
@simozhan
simozhan force-pushed the add_association_has_many branch from f06cb84 to 56de980 Compare January 16, 2018 05:47
@simozhan
simozhan force-pushed the add_association_has_many branch from 7d2b5b1 to a42c614 Compare January 25, 2018 08:58
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.

1 participant