-
Notifications
You must be signed in to change notification settings - Fork 354
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
Support custom row classes for manager #3004
Conversation
🚀 Deployed on https://deploy-preview-3004--moor.netlify.app |
This may make it harder to add features later, but nothing impossible. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like tests are failing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some small assert was causing bugs, LGTM now
Thanks for taking a look and fixing the assert! |
This adds support for generating manager classes for tables with existing row classes. At the moment, the only thing I had to change was removing the
DataClass
generic bound in the runtime.@dickermoshe, can you check whether you need the
extends DataClass
bound for future manager changes? If not I think this sounds like a fairly easy way to port manager code over to more tables.Closes #2989.