volto icon indicating copy to clipboard operation
volto copied to clipboard

Enhancement suggestion: Show NotFound when a control panel is not found

Open alexandreIFB opened this issue 6 months ago • 9 comments

Currently, when accessing a non-existent control panel URL (e.g., /controlpanel/does-not-exist), the Controlpanel component returns an empty <div /> if the panel is not found. This provides no visual feedback to the user about the error.

Problem

  • Users who type an incorrect URL or try to access a removed panel receive no indication that the panel does not exist.
  • This can be confusing, as the screen remains blank with no error message.

Proposal

Change the component to explicitly render the NotFound screen (<NotFound />) when the panel is not found, instead of returning an empty <div />.

Benefits:

  • Provides clear feedback to the user when a panel does not exist.
  • Improves the user experience and usability of the system.

alexandreIFB avatar Jun 24 '25 14:06 alexandreIFB

hello sir. Is this issue is still open ?

Digvijay-x1 avatar Jul 25 '25 19:07 Digvijay-x1

@stevepiercy I think no one is working on this issue. Can i start working on it?

VibhuGupta-dev avatar Nov 28 '25 18:11 VibhuGupta-dev

@VibhuGupta-dev If don't have an open PR and no one is assigned to the issue, you can simply start working on it and open a PR. No request is necessary.

wesleybl avatar Nov 28 '25 20:11 wesleybl

@VibhuGupta-dev But in this case, there is already an open PR: https://github.com/plone/volto/pull/7218

However, the issue is not referenced in the PR description; I will fix it.

You can talk to @shashank0470

wesleybl avatar Nov 28 '25 20:11 wesleybl

Hey @shashank0470

Since this PR has been inactive for a while and the requested changes seem to be pending because the updated code wasn't pushed, I wanted to check whether you're still working on this.

I have implemented a complete working fix for the same issue in PR #7663 (including the routing loading condition and the NotFound behavior). I'm happy to collaborate with you — we can merge the approaches if needed.

If you prefer, I can also take over this PR and push the required changes here so that it can move forward. Just let me know what works best for you.

Thanks!

VibhuGupta-dev avatar Nov 28 '25 20:11 VibhuGupta-dev

Thanks for checking! @VibhuGupta-dev
I haven’t been able to complete the pending updates.
If you’d like to commit the required changes directly to this PR, please go ahead — that works for me.
Thanks for helping move it forward!

shashank0470 avatar Nov 28 '25 20:11 shashank0470

I noticed the issue is still present. I’ve implemented a fix on my side, would it be helpful if I opened a PR?

iyush05 avatar Dec 11 '25 06:12 iyush05

@VibhuGupta-dev can help you on this if he is not working on it.

shashank0470 avatar Dec 11 '25 07:12 shashank0470

We have 3 open PRs for this issue now. @iyush05 please read and follow First-time contributors, especially Things not to do, Contributing to Plone, and Contributing to Volto.

stevepiercy avatar Dec 11 '25 11:12 stevepiercy