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

After deleting all sub-pages it's not possible to delete the parent page #3144

Open
juliankehne opened this issue Oct 22, 2024 · 4 comments
Open
Labels
bug Something isn't working prio: low Not urgent, can be resolved in the distant future.
Milestone

Comments

@juliankehne
Copy link

juliankehne commented Oct 22, 2024

Describe the Bug

Its not directly possible to delete a page after deleting its sub-pages. The parent page first has to be updated or the page tree needs to be refreshed (e.g. by moving a page or creating a new page) before the page can be deleted.

Steps to Reproduce

  1. Go to 'Pages'
  2. Create a new page
  3. Create a second page as a sub-page for the first page
  4. Go back to 'Pages'
  5. Delete the sub-page
  6. Delete the parent page
  7. See that "Delete" icon is inactive (screenshot below)

Expected Behavior

"Delete" button should be clickable without having to update the page or page tree after deleting the sub-pages

Actual Behavior

After deleting the last sub-page of a page the "Delete" icon is greyed out and cannot be clicked. It only becomes available once page is updated or the page tree was refreshed, e.g. by moving a page or creating a new one.

Additional Information

Found in Version: 2024.10.12-alpha

cannot delete page without subpage

@JoeyStk JoeyStk added the bug Something isn't working label Oct 22, 2024
@JoeyStk JoeyStk added this to the 25Q3 milestone Oct 22, 2024
@JoeyStk JoeyStk added the prio: low Not urgent, can be resolved in the distant future. label Oct 22, 2024
@david-venhoff
Copy link
Member

Cannot reproduce this locally, but I can on the test server. Since I have redis disabled locally, it is probably a redis caching issue.

@osmers
Copy link

osmers commented Oct 22, 2024

I mentioned during the call that I could reproduce it on the test system, even so far that I could not delete the page, even after refreshing - I found the problem with that @PeterNerlich: there were archived pages still attributed to this parent page so it appeared empty in the page tree but for system logic was not, bcs the archived pages are connected to it. I only saw that once I opened the page and tried to delete it there... so probably it is what David says :)

@PeterNerlich
Copy link
Collaborator

there were archived pages still attributed to this parent page so it appeared empty in the page tree but for system logic was not

That makes sense. The popup text does mention archived sub pages, but I think the communication here can be dramatically improved. We should at least make the popup more dynamic and tell exactly whether the page has archived sub pages or not.

This is a completely new idea I don't think we have anywhere else in the system, but what do you think about highlighting the Archived pages button when hovering over the disabled delete icon of a page that has archived children? And maybe even blinking it twice when the disabled delete icon is clicked?

@osmers
Copy link

osmers commented Oct 23, 2024

We could think about that but definitely low prio as it is a problem that only affects us and not the municipalities. I think we have bigger issues to work on right now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working prio: low Not urgent, can be resolved in the distant future.
Projects
None yet
Development

No branches or pull requests

5 participants