-
Notifications
You must be signed in to change notification settings - Fork 134
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
Allow plugins to override tables #1093
Conversation
120e5ce
to
7e82e94
Compare
7e82e94
to
f32969c
Compare
Hey Joaquim, this PR works well in all our cases. Thanks. |
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.
Very good. I think the API is nice.
I left some notes for your consideration.
Would you mind rebasing on main? (there's quite a few changes merged since this was done)
4ad647b
to
29f13dc
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.
Thanks for those updates.
I resolved a bunch of convos, and left a few more notes for your consideration.
It's maybe worth a rebase against main.
29f13dc
to
0b249b6
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.
The part after the headlamp- prefix needs to be described. Added some suggested docs in one convo.
0b249b6
to
05fea7a
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.
🎉 thanks!
All the convos are resolved. Just waiting for tests.
When they don't use the resourceClass argument, they don't get an ID automatically set.
05fea7a
to
bfd5ec8
Compare
Love this PR. It means lot to customization. |
This PR adds the changes and an example plugin on modifying existing tables throughout Headlamp.
The way this is done is by having ResourceTables identified with a new ID (or automatically done when they receive the resourceClass argument) and then plugins can declare their processors and use this ID within the processors' function to decide whether to modify the columns that the table receives.
fixes #1044