-
Notifications
You must be signed in to change notification settings - Fork 26
🐛 Bug Report: [ThemeProvider - mui v5 upgrade] Error opening Announcements page #472
Comments
@stephenglass - will look into this tomorrow. This has something to do with the upgrade to mui v5. What version is your instance on, and are you using a custom theme? |
Thanks! I'm using Backstage 1.31.3 and still using MUI v4 with a custom theme. |
@stephenglass, have you migrated to use the |
Yes, we're using |
I haven't been able to reproduce this locally. We use a custom theme and a mixute of material 4/5 internally. |
I upgraded our Backstage version from 1.31.3 to 1.32.1, dropped the announcements database, restarted and now it's working. I'm honestly at a loss on what the issue was or how it was resolved. Perhaps it was some bad dependency resolution that was fixed by the upgrade, not sure. |
When updating our Backstage version to 1.32.1 and The errors I see are different (see Details): TypeError
undefined is not an object (evaluating 'theme.palette.text')
Call Stack
undefined
:undefined:undefined
create
:undefined:undefined
attach
:undefined:undefined
undefined
:undefined:undefined
useSynchronousEffect
:undefined:undefined
useStyles
:undefined:undefined
AnnouncementsGrid
:undefined:undefined
renderWithHooks
:undefined:undefined
mountIndeterminateComponent
:undefined:undefined
callCallback
:undefined:undefined ERROR
undefined is not an object (evaluating 'theme.palette.text')
@
create@
attach@
@
useSynchronousEffect@
useStyles@
AnnouncementsGrid@
renderWithHooks@
mountIndeterminateComponent@
callCallback@
runTask@
invokeTask@
invokeTask@
globalCallback@
dispatchEvent@[native code]
invokeGuardedCallbackDev@
invokeGuardedCallback@
beginWork$1@
performUnitOfWork@
workLoopConcurrent@
renderRootConcurrent@
performConcurrentWorkOnRoot@
workLoop@
flushWork@
performWorkUntilDeadline@ Despite the different errors, it appears the core issue is similar, which is that I noticed that the announcements plugin's migration to MUI v5 seemed to remove the AFAIU, the |
I am no longer seeing this issue after updating to Backstage v1.32.5 and announcements v0.11.2. Edit: Added the environment details below where I am no longer experiencing the issue. Environment details/backstage-cli info dump
OS: Darwin 23.6.0 - darwin/arm64
node: v18.19.1
yarn: 1.22.22
cli: 0.28.2 (installed)
backstage: 1.32.5
Dependencies:
@backstage/app-defaults 1.5.12
@backstage/backend-app-api 0.8.0, 1.0.1
@backstage/backend-common 0.23.3, 0.24.1, 0.25.0
@backstage/backend-defaults 0.4.1, 0.5.1, 0.5.2
@backstage/backend-dev-utils 0.1.5
@backstage/backend-openapi-utils 0.2.0
@backstage/backend-plugin-api 0.7.0, 0.8.1, 1.0.1
@backstage/backend-tasks 0.5.27, 0.6.1
@backstage/backend-test-utils 1.0.1, 1.0.2
@backstage/catalog-client 1.7.1
@backstage/catalog-model 1.7.0
@backstage/cli-common 0.1.14
@backstage/cli-node 0.2.9
@backstage/cli 0.28.2
@backstage/config-loader 1.9.1
@backstage/config 1.2.0
@backstage/core-app-api 1.15.1
@backstage/core-compat-api 0.3.1
@backstage/core-components 0.13.10, 0.14.10, 0.15.1
@backstage/core-plugin-api 1.10.0
@backstage/dev-utils 1.1.2
@backstage/errors 1.2.4
@backstage/eslint-plugin 0.1.10
@backstage/frontend-app-api 0.10.0
@backstage/frontend-defaults 0.1.1
@backstage/frontend-plugin-api 0.8.0, 0.9.0
@backstage/frontend-test-utils 0.2.1
@backstage/integration-aws-node 0.1.12
@backstage/integration-react 1.2.0
@backstage/integration 1.15.1
@backstage/plugin-api-docs 0.11.11
@backstage/plugin-app-backend 0.3.76
@backstage/plugin-app-node 0.1.26
@backstage/plugin-app 0.1.1
@backstage/plugin-auth-backend-module-atlassian-provider 0.3.1
@backstage/plugin-auth-backend-module-auth0-provider 0.1.1
@backstage/plugin-auth-backend-module-aws-alb-provider 0.2.1
@backstage/plugin-auth-backend-module-azure-easyauth-provider 0.2.1
@backstage/plugin-auth-backend-module-bitbucket-provider 0.2.1
@backstage/plugin-auth-backend-module-bitbucket-server-provider 0.1.1
@backstage/plugin-auth-backend-module-cloudflare-access-provider 0.3.1
@backstage/plugin-auth-backend-module-gcp-iap-provider 0.3.1
@backstage/plugin-auth-backend-module-github-provider 0.2.1
@backstage/plugin-auth-backend-module-gitlab-provider 0.2.1
@backstage/plugin-auth-backend-module-google-provider 0.2.1
@backstage/plugin-auth-backend-module-microsoft-provider 0.2.1
@backstage/plugin-auth-backend-module-oauth2-provider 0.3.1
@backstage/plugin-auth-backend-module-oauth2-proxy-provider 0.2.1
@backstage/plugin-auth-backend-module-oidc-provider 0.3.1
@backstage/plugin-auth-backend-module-okta-provider 0.1.1
@backstage/plugin-auth-backend-module-onelogin-provider 0.2.1
@backstage/plugin-auth-backend 0.23.1
@backstage/plugin-auth-node 0.4.17, 0.5.3
@backstage/plugin-auth-react 0.1.7
@backstage/plugin-catalog-backend-module-aws 0.4.4
@backstage/plugin-catalog-backend-module-backstage-openapi 0.4.1
@backstage/plugin-catalog-backend-module-github 0.7.6
@backstage/plugin-catalog-backend-module-openapi 0.2.3
@backstage/plugin-catalog-backend 1.27.1
@backstage/plugin-catalog-common 1.1.0
@backstage/plugin-catalog-graph 0.4.11
@backstage/plugin-catalog-node 1.13.1
@backstage/plugin-catalog-react 1.14.0
@backstage/plugin-catalog 1.24.0
@backstage/plugin-events-backend 0.3.13
@backstage/plugin-events-node 0.3.8, 0.4.1, 0.4.4
@backstage/plugin-home-react 0.1.18
@backstage/plugin-home 0.8.0
@backstage/plugin-kubernetes-common 0.8.3
@backstage/plugin-org 0.6.31
@backstage/plugin-permission-backend 0.5.50
@backstage/plugin-permission-common 0.8.1
@backstage/plugin-permission-node 0.8.4
@backstage/plugin-permission-react 0.4.27
@backstage/plugin-proxy-backend 0.5.7
@backstage/plugin-scaffolder-common 1.5.6
@backstage/plugin-scaffolder-node 0.4.8
@backstage/plugin-search-backend-module-catalog 0.2.4
@backstage/plugin-search-backend-module-elasticsearch 1.6.1
@backstage/plugin-search-backend-module-techdocs 0.3.1
@backstage/plugin-search-backend-node 1.3.3, 1.3.4
@backstage/plugin-search-backend 1.6.1
@backstage/plugin-search-common 1.2.14
@backstage/plugin-search-react 1.8.1
@backstage/plugin-search 1.4.18
@backstage/plugin-signals-node 0.1.12
@backstage/plugin-signals-react 0.0.5, 0.0.6
@backstage/plugin-techdocs-backend 1.11.1
@backstage/plugin-techdocs-common 0.1.0
@backstage/plugin-techdocs-module-addons-contrib 1.1.16
@backstage/plugin-techdocs-node 1.12.12
@backstage/plugin-techdocs-react 1.2.9
@backstage/plugin-techdocs 1.11.0
@backstage/plugin-user-settings-common 0.0.1
@backstage/plugin-user-settings 0.8.14
@backstage/release-manifests 0.0.11
@backstage/test-utils 1.7.0
@backstage/theme 0.5.7, 0.6.0
@backstage/types 1.1.1
@backstage/version-bridge 1.0.10 |
I'm on Backstage v1.32.5 and Announcements v0.11.0 - v0.11.2 are all showing this issue for me. The last stable one is v0.10.5. |
@jvanalstyne-mdsol could you share the output of I'm also having this issue after the mui v5 update. I suppose me as the others having some mismatch in the version between the various plugins. |
Sure thing - here is the output:
Hope this helps!! |
same as @lynettelopez it works for me only with |
Any idea how to work around this ? |
I think you should revert the MUI5 update. See this RFC: backstage/backstage#27726 |
I've been out again the past two weeks - What about this message @drodil?
|
Hi @kurtaking Yes, there was a discussion about this and it's not necessary to revert MUI5 changes (if they work 😆 ). I reverted my own plugin as it wasn't really functional with MUI5. So I think it's your call! |
I think the best course of action right now is reverting. |
Working on this PR to revert back to MUI 4. |
Looking for reviews on the PR if anyone is interested. |
Has this resolved itself with the revert to mui 4? |
📜 Description
Going to the announcements route (
<AnnouncementsPage/>
) shows an error and unable to load the page.Noticed this when upgrading
@procore-oss/backstage-plugin-announcements
from0.10.0
->0.11.1
. The last working version is0.10.5
.Error suggests that
AnnouncementsPage
component need to be wrapped in aThemeProvider
to be able to access the theme params in themakeStyles
.👍 Expected behavior
Should be able to open the announcements page
👎 Actual Behavior with Screenshots
Console error logs:
👟 Reproduction steps
Go to the route serving the
<AnnouncementsPage/>
component📃 Provide the context for the Bug.
Unable to display the announcements page
🖥️ Your Environment
Backstage v1.31.3
👀 Have you spent some time to check if this bug has been raised before?
Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else
The text was updated successfully, but these errors were encountered: