We have moved to GitHub Issues
Created by Jeavon Leopold 20 Jun 2016, 12:06:03 Updated by Shannon Deminick 05 Aug 2016, 11:07:16Tags: PR
I have updated UmbracoFileSystemProviders.Azure to include the new IconUrl for testing, you can take the Zip from https://ci.appveyor.com/project/JamesSouth/umbracofilesystemproviders-azure/build/artifacts
Result is so tiny but so nice :)
ah, this is great, i had thought we'd have to make an Icon Url field - looks like we don't need that. I'll pull in and update where applicable, thanks!
You had already added it to the Installer just needed to get the data to the UI via the API controller.
Now how about adding a Icon field to manage package on Our, ah, that's a different tracker :)
Yup, figuring all this out now :) The IconUrl will be a field when you create a package in the back office in 7.5+ and will be part of it's manifest.
Hi Just Looking I can't see this PR in the Dev-v7 Branch anymore? https://github.com/umbraco/Umbraco-CMS/blob/dev-v7/src/Umbraco.Web.UI.Client/src/views/packager/views/install-local.html#L72
vm.localPackage.icon when it should be
I am reasonably sure I am looking at the right file
also i think repo.html and installed.html also need icon changing to iconUrl, but i wanted to make sure i was looking the right place before doing anything
fairly sure we've just streamlined the naming of the icon property - you can retest if you run from the latest vs solution in dev-v7 branch. I'll see what I can find today as well.
I think maybe this has been mistakenly merged out, the other changes are still in place so there will be a mismatch in the model...
Hrm, does seem odd! I'll fix this up
all fixed up in rev: 3e932ecf3fdd527795b805cc319dd569e8a3bea6
Backwards Compatible: True
Fix Submitted: Pull request
Affected versions: 7.5.0
Due in version: 7.5.0
Sprint: Sprint 36