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

V8: Don't "Create matching template" before the content type is created #4890

Conversation

kjac
Copy link
Contributor

@kjac kjac commented Mar 6, 2019

Prerequisites

  • I have added steps to test this contribution in the description below

Description

When creating a "Document Type without a template", you're given the option to "Create matching template" even before the content type is created. Attempting to do so causes a failure, albeit not very well handled:

create-matching-template-fail

This PR simply removes the "Create matching template" button until the content type is actually created:

create-matching-template-ok

@emmaburstow
Copy link
Contributor

Ah brilliant! I'll take a look at this one and let you know if we need anything.

Em

Copy link
Contributor

@emmaburstow emmaburstow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect. Thanks @kjac

@zpqrtbnk zpqrtbnk changed the base branch from dev-v8 to v8/dev March 30, 2019 10:25
@nul800sebastiaan nul800sebastiaan merged commit 6bfcc7b into umbraco:v8/dev Apr 30, 2019
@nul800sebastiaan
Copy link
Member

Makes total sense! ⭐️

@kjac kjac deleted the v8-fix-create-matching-template-without-content-type branch April 30, 2019 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants