{"action":"created","issue":{"url":"https://api.github.com/repos/TryGhost/Ghost/issues/6320","repository_url":"https://api.github.com/repos/TryGhost/Ghost","labels_url":"https://api.github.com/repos/TryGhost/Ghost/issues/6320/labels{/name}","comments_url":"https://api.github.com/repos/TryGhost/Ghost/issues/6320/comments","events_url":"https://api.github.com/repos/TryGhost/Ghost/issues/6320/events","html_url":"https://github.com/TryGhost/Ghost/pull/6320","id":126003849,"number":6320,"title":"[WIP] Editor/PSM refactor","user":{"login":"kevinansfield","id":415,"avatar_url":"https://avatars.githubusercontent.com/u/415?v=3","gravatar_id":"","url":"https://api.github.com/users/kevinansfield","html_url":"https://github.com/kevinansfield","followers_url":"https://api.github.com/users/kevinansfield/followers","following_url":"https://api.github.com/users/kevinansfield/following{/other_user}","gists_url":"https://api.github.com/users/kevinansfield/gists{/gist_id}","starred_url":"https://api.github.com/users/kevinansfield/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kevinansfield/subscriptions","organizations_url":"https://api.github.com/users/kevinansfield/orgs","repos_url":"https://api.github.com/users/kevinansfield/repos","events_url":"https://api.github.com/users/kevinansfield/events{/privacy}","received_events_url":"https://api.github.com/users/kevinansfield/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-01-11T18:13:53Z","updated_at":"2016-06-06T09:14:09Z","closed_at":"2016-06-06T09:14:09Z","pull_request":{"url":"https://api.github.com/repos/TryGhost/Ghost/pulls/6320","html_url":"https://github.com/TryGhost/Ghost/pull/6320","diff_url":"https://github.com/TryGhost/Ghost/pull/6320.diff","patch_url":"https://github.com/TryGhost/Ghost/pull/6320.patch"},"body":"closes #6315 (alternative fix)\r\n\r\n- add `ember-one-way-controls` addon that provides true one-way bound inputs\r\n- move title->slug generation from PSM to main editor controller, remove use of observers, simplify promise handling\r\n  - also removes attempt at local slug generation if no slug exists when saving as the server will generate and return the slug anyway\r\n  - lays the groundwork for a sensible fix for #5062 (unpublished title not auto-updating slug)\r\n\r\nTODO:\r\n- make all inputs one-way bound with any processing attached to update actions\r\n- investigate need for scratch values, especially when they are on the model and not local to the controller\r\n- standardise on either immediately saving on PSM changes or making changes locally and only saving when \"Save\" is pushed\r\n- test all the things"},"comment":{"url":"https://api.github.com/repos/TryGhost/Ghost/issues/comments/223907009","html_url":"https://github.com/TryGhost/Ghost/pull/6320#issuecomment-223907009","issue_url":"https://api.github.com/repos/TryGhost/Ghost/issues/6320","id":223907009,"user":{"login":"kevinansfield","id":415,"avatar_url":"https://avatars.githubusercontent.com/u/415?v=3","gravatar_id":"","url":"https://api.github.com/users/kevinansfield","html_url":"https://github.com/kevinansfield","followers_url":"https://api.github.com/users/kevinansfield/followers","following_url":"https://api.github.com/users/kevinansfield/following{/other_user}","gists_url":"https://api.github.com/users/kevinansfield/gists{/gist_id}","starred_url":"https://api.github.com/users/kevinansfield/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kevinansfield/subscriptions","organizations_url":"https://api.github.com/users/kevinansfield/orgs","repos_url":"https://api.github.com/users/kevinansfield/repos","events_url":"https://api.github.com/users/kevinansfield/events{/privacy}","received_events_url":"https://api.github.com/users/kevinansfield/received_events","type":"User","site_admin":false},"created_at":"2016-06-06T09:14:09Z","updated_at":"2016-06-06T09:14:09Z","body":"Closing this as the `one-way-controls` work in this PR is now on-going in [Ghost-Admin](https://github.com/TryGhost/Ghost-Admin/pull/46). There is still room for a refactor of the Editor/PSM code on top of that work but it will be easier to build upon once that is merged."}}
{"actor":{"display_login":"kevinansfield"}}