Skip to content

fix(Subpath): dispatch async actions for controlpanels on /controlpanels - #8259

Open
nileshgulia1 wants to merge 5 commits into
mainfrom
subpath-controlpanel
Open

fix(Subpath): dispatch async actions for controlpanels on /controlpanels#8259
nileshgulia1 wants to merge 5 commits into
mainfrom
subpath-controlpanel

Conversation

@nileshgulia1

Copy link
Copy Markdown
Member

closes #8258

@nileshgulia1
nileshgulia1 marked this pull request as ready for review May 20, 2026 14:50
@nileshgulia1
nileshgulia1 requested a review from wesleybl May 21, 2026 14:37
@nileshgulia1

Copy link
Copy Markdown
Member Author

@wesleybl Does this ever happened with any of your prefixed site as well?

Comment thread packages/volto/news/8258.bugfix Outdated
Comment thread packages/volto/src/helpers/Site/index.js Outdated
nileshgulia1 and others added 2 commits May 22, 2026 14:51
Co-authored-by: Piero Nicolli <pnicolli@users.noreply.github.com>
@nileshgulia1
nileshgulia1 requested a review from pnicolli May 22, 2026 09:24
@pnicolli

Copy link
Copy Markdown
Contributor

Looks good to me but I don't have good examples to test the bugfix on, so I would recommend also waiting for @wesleybl to review

@wesleybl

Copy link
Copy Markdown
Member

@nileshgulia1 I don't understand why this is necessary. These actions are already added here:

export const Controlpanels = asyncConnect([
{
key: 'controlpanels',
promise: async ({ store: { dispatch } }: any) =>
await dispatch(listControlpanels()),
},
{
key: 'systemInformation',
promise: async ({ store: { dispatch } }: any) =>
await dispatch(getSystemInformation()),
},
])(LoadableControlpanels);

I couldn't reproduce the problem locally. Even at https://demo.plone.org/controlpanel, I managed to reproduce it once, but then I couldn't anymore. Something is strange.

@davisagli davisagli added this to the 19.x.x milestone May 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

volto 19(Subpath feature): ssr calls fails in /controlpanel

4 participants