Skip to content
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

migrate the Kubernetes plugin to mui 5 #27572

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

sebalaini
Copy link
Contributor

@sebalaini sebalaini commented Nov 10, 2024

Hey, I just made a Pull Request!

Migrate the Kubernetes plugin to mui 5.

The reason for using import Grid from '@mui/material/Unstable_Grid2' is to reduce tech debt in the future, in mui 6 there is the following warning:

The Grid component has been deprecated. Please use Grid v2 instead.

Screenshot 2024-11-10 at 15 26 51 Screenshot 2024-11-10 at 15 27 18 Screenshot 2024-11-10 at 15 26 23

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)
  • All your commits have a Signed-off-by line in the message. (more info)

Signed-off-by: sebalaini <sebastianolaini@gmail.com>
@github-actions github-actions bot added the area:kubernetes Related to the Kubernetes Project Area - not deploying Backstage with k8s. label Nov 10, 2024
@backstage-goalie
Copy link
Contributor

backstage-goalie bot commented Nov 10, 2024

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage/plugin-kubernetes-react plugins/kubernetes-react patch v0.5.0-next.2
@backstage/plugin-kubernetes plugins/kubernetes patch v0.12.0-next.3

@sebalaini
Copy link
Contributor Author

@mclarke47 @Rugvip I can't add reviewers so I'm tagging you in a comment as I know you are reviewing the K8s plugin PRs :)

Signed-off-by: sebalaini <sebastianolaini@gmail.com>
"@material-ui/core": "^4.9.13",
"@material-ui/icons": "^4.11.3",
"@material-ui/lab": "^4.0.0-alpha.61",
"@material-ui/core": "^4.12.2",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this required still?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sadly, I had to revert some changes; you can see this commit because some tests were failing with TypeError: theme.spacing is not a function, it can be seen here

Signed-off-by: sebalaini <sebastianolaini@gmail.com>
Signed-off-by: sebalaini <sebastianolaini@gmail.com>
@awanlin
Copy link
Collaborator

awanlin commented Nov 25, 2024

Hi @sebalaini, given this #27726, I'm wondering if we should hold off on this for now.

@LegendSebastianoL
Copy link

hey @awanlin sure :) I didn't know about that RFC, though I wonder if that RFC should be mentioned in the documentation https://backstage.io/docs/tutorials/migrate-to-mui5 so that people can either migrate or wait to see what would be the decision :)

@awanlin
Copy link
Collaborator

awanlin commented Nov 25, 2024

hey @awanlin sure :) I didn't know about that RFC, though I wonder if that RFC should be mentioned in the documentation https://backstage.io/docs/tutorials/migrate-to-mui5 so that people can either migrate or wait to see what would be the decision :)

No worries, I mean the RFC came after you had submitted the PR, just wanted you to be aware. Your suggestion about a link in the migration guide makes a lot of sense, can you run with that?

@LegendSebastianoL
Copy link

hey @awanlin sorry for the delay, do you want me to add a link to the RFC in the doc and open a PR?

@drodil
Copy link
Contributor

drodil commented Nov 27, 2024

@LegendSebastianoL see #27849

@LegendSebastianoL
Copy link

@LegendSebastianoL see #27849

Noted :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:kubernetes Related to the Kubernetes Project Area - not deploying Backstage with k8s.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants