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

Table mapping support for one to many #32

Merged
merged 3 commits into from May 17, 2017

Conversation

Projects
None yet
2 participants
@Matthewrbowker
Member

Matthewrbowker commented May 15, 2017

This code allows us to map different tables based on the labs-specific index.

It also removes some unnecessary code from the table mapping - an exception that should be handled in the map itself.

Fixes T164556

@Matthewrbowker Matthewrbowker requested review from samwilson and MusikAnimal May 15, 2017

@MusikAnimal

This comment has been minimized.

Member

MusikAnimal commented May 17, 2017

Looks good!

@MusikAnimal MusikAnimal merged commit 7afccb7 into master May 17, 2017

1 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build failed
Details
Scrutinizer 2 new issues, 1 updated code elements
Details

@Matthewrbowker Matthewrbowker deleted the T164556 branch May 17, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment