Skip to content
This repository has been archived by the owner on Dec 13, 2021. It is now read-only.

Upgrades Umbraco dependency to v7.7.0 #22

Merged
merged 9 commits into from
Apr 11, 2018
Merged

Upgrades Umbraco dependency to v7.7.0 #22

merged 9 commits into from
Apr 11, 2018

Conversation

leekelleher
Copy link
Contributor

@leekelleher leekelleher commented Apr 1, 2018

This PR is targeting the "dev-v2" branch, I've made this as a separate branch/PR, so to not impact the main "develop" branch and reduce noise on any following "v2" PRs, (see #23)

@leekelleher leekelleher added this to the 2.0.0 milestone Apr 1, 2018
@leekelleher leekelleher self-assigned this Apr 1, 2018
This is done as a separate branch/PR, so to reduce the noise of following "v2" PRs.
Reworks how the Content Type picker overlay works.
Targets the overlay to the "right" panel and enables Content Type selection to follow the same pattern as creating regular content nodes.
When a Content Type is selected, any Content Templates (Blueprints) will be displayed and can be selected.

Underlying updates have been made to the InnerContent resource & service scripts - these may be breaking changes, though I have only tested using Stacked Content, which (at present) has no side effects.
@leekelleher
Copy link
Contributor Author

I am wondering whether we could jump to v7.8+ ... then we could remove the version checker in JS for removing the "Generic Properties" tab. No biggie, only a thought.

leekelleher and others added 7 commits April 10, 2018 12:01
for consistency
It makes sense in the context of the `getContentType` function.

#23 (comment)
Removed the unnecessary empty/null values that are were being passed in,
(apart from one place where we have to)

#23 (comment)
- Clarified a TODO note
- Removed the "tabs" field, as it isn't being used by IC (nor SC)
- Renamed JS local variable
Adds support for Content Templates (aka Blueprints)
@leekelleher leekelleher merged commit ea400a5 into dev-v2 Apr 11, 2018
@leekelleher leekelleher deleted the upgrade-umb77 branch April 11, 2018 10:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

1 participant