-
Notifications
You must be signed in to change notification settings - Fork 56
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
Read-only views of joined data #18
Comments
This is a nice feature since in SQL, an entity can be divided into different tables in order to avoid duplication |
Hey @greentfrapp , playing around with this today. I think it's a good first pass, but can we make configuring a Join Attribute more explicit? I get why you might not want to introduce a separate config object, but I think it would help someone know what's going on just by looking at the code or config - right now I have to rely on the example or dig into the code. A separate object might also prevent having to do all the regex and make it easier to add more options in the future. Just my 2 cents :) let me know if you wanna discuss |
Woops sorry @OneFishTwoFishRedFish-JFish I think this was automatically closed when I merged a PR that said "fixed 18" Reopening for discussion - will take a closer look! |
No description provided.
The text was updated successfully, but these errors were encountered: