-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
refactor(table): rename MatTableDataSource to ClientArrayTableDataSource #15432
Conversation
ac4664a
to
705ec29
Compare
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.
LGTM
@andrewseguin should also approve this before it goes in (since he might disagree)
Renames the `MatTableDataSource` to `ClientArrayTableDataSource` to make it a bit clearer what it's supposed to be used for. Fixes angular#14378.
705ec29
to
1deb759
Compare
Added to 11.0.0 milestone |
Needs rebase |
I'm closing this since it's pretty breaking and way out of date. We can recreate it if we still think it's a good idea. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Renames the
MatTableDataSource
toClientArrayTableDataSource
to make it a bit clearer what it's supposed to be used for.Fixes #14378.