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.

Sign up for GitHub

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

Jump to bottom

Possible to delete templates with children #16443

Closed
bergmania opened this issue May 29, 2024 · 2 comments
Closed

Possible to delete templates with children #16443

bergmania opened this issue May 29, 2024 · 2 comments
Labels
area/backend release/15.1.0

Comments

@bergmania
Copy link
Member

bergmania commented May 29, 2024 •
edited by azure-devops-sync bot
Loading

Which Umbraco version are you using? (Please write the exact version, example: 10.1.0)

V14 RC5

Bug summary

It is possible to delete (and I have the option in the client) a template that has children.

image

Specifics

No response

Steps to reproduce

  1. Create a template
  2. Create another template below the firstly created
  3. Try to delete the template with children

Expected result / actual result

It should not be possible in the management api and I should not have the option in the client

This item has been added to our backlog AB#44595
@iOvergaard iOvergaard added the state/sprint-candidate We're trying to get this in a sprint at HQ in the next few weeks label Oct 2, 2024
@leekelleher leekelleher self-assigned this Oct 23, 2024
@leekelleher leekelleher mentioned this issue Oct 23, 2024
Merged
1 task
@leekelleher
Copy link
Member

Fix for the frontend is available in PR: umbraco/Umbraco.CMS.Backoffice#2479.

@leekelleher leekelleher removed their assignment Oct 23, 2024
bergmania added a commit that referenced this issue Nov 15, 2024
@bergmania
https://github.com/umbraco/Umbraco-CMS/issues/16443
a75f310 
Also ensure master templates cannot be deleted
@bergmania bergmania mentioned this issue Nov 15, 2024
Merged
@bergmania
Copy link
Member Author

Fixed in #17539

@bergmania bergmania added release/15.1.0 and removed state/sprint-candidate We're trying to get this in a sprint at HQ in the next few weeks type/bug state/reproduced affected/v14 affected/v15 labels Nov 15, 2024
bergmania added a commit that referenced this issue Nov 16, 2024
@bergmania @nikolajlauridsen
Fixes to templates: Do not allow deletion of masters and ensure file …
c846633 
…names are pascal case (#17539)

* Make template file names start with capical character

* #16443

Also ensure master templates cannot be deleted

* Fix test

* Do not check breaking changes in integration tests

---------

Co-authored-by: nikolajlauridsen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/backend release/15.1.0
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@leekelleher @iOvergaard @bergmania