{"action":"closed","issue":{"url":"https://api.github.com/repos/Novicell/novicell-mapbuilder/issues/4","repository_url":"https://api.github.com/repos/Novicell/novicell-mapbuilder","labels_url":"https://api.github.com/repos/Novicell/novicell-mapbuilder/issues/4/labels{/name}","comments_url":"https://api.github.com/repos/Novicell/novicell-mapbuilder/issues/4/comments","events_url":"https://api.github.com/repos/Novicell/novicell-mapbuilder/issues/4/events","html_url":"https://github.com/Novicell/novicell-mapbuilder/issues/4","id":169741917,"number":4,"title":"Some adjustments for project","user":{"login":"bjarnef","id":2919859,"avatar_url":"https://avatars.githubusercontent.com/u/2919859?v=3","gravatar_id":"","url":"https://api.github.com/users/bjarnef","html_url":"https://github.com/bjarnef","followers_url":"https://api.github.com/users/bjarnef/followers","following_url":"https://api.github.com/users/bjarnef/following{/other_user}","gists_url":"https://api.github.com/users/bjarnef/gists{/gist_id}","starred_url":"https://api.github.com/users/bjarnef/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bjarnef/subscriptions","organizations_url":"https://api.github.com/users/bjarnef/orgs","repos_url":"https://api.github.com/users/bjarnef/repos","events_url":"https://api.github.com/users/bjarnef/events{/privacy}","received_events_url":"https://api.github.com/users/bjarnef/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Novicell/novicell-mapbuilder/labels/invalid","name":"invalid","color":"e6e6e6"}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-08-06T11:07:35Z","updated_at":"2016-08-06T11:27:02Z","closed_at":"2016-08-06T11:27:02Z","body":"Can we include an embed SQL CE database in `/App_Data` folder in this project, like we do in Analytics project over here and note login in readme? https://github.com/warrenbuckley/Analytics/tree/develop/Analytics.Site/App_Data\r\n\r\nFurthermore I think there is no reason to include `/App_Browsers` and `/Config/splashes` folder in library: https://github.com/Novicell/novicell-mapbuilder/tree/master/MapBuilder.Library\r\n\r\nIn an single line you are using you own localize directive https://github.com/Novicell/novicell-mapbuilder/blob/4a3c0ecfaeaa2a540ddb1f78e7dba6627a911feb/MapBuilder.Website/App_Plugins/NcMapBuilder/BackOffice/MapBuilderTree/edit.html#L20 ... is there a reason why not just use `<localize>` element directive?\r\n\r\nIf you want to limit support for this project to Umbraco 7.4+ it would probably be easier to maintain and you can use all the new cool directives/compents, e.g. you can consider to use `umb-table` for this part: https://github.com/Novicell/novicell-mapbuilder/blob/4a3c0ecfaeaa2a540ddb1f78e7dba6627a911feb/MapBuilder.Website/App_Plugins/NcMapBuilder/BackOffice/MapBuilderTree/edit.html#L22-L53\r\n\r\nMost of the stuff is documented here https://our.umbraco.org/apidocs/ui/#/api .. but some might be missing, so it might be better to stuff source code although there might be some changes for v7.5.0\r\n\r\nE.g. something like this https://github.com/umbraco/Umbraco-CMS/blob/75c2b07ad3a093b5b65b6ebd45697687c062f62a/src/Umbraco.Web.UI.Client/src/views/media/edit.html to build to view with tabs, header, footer with buttons."}}
{"actor":{"display_login":"bjarnef"}}