{"action":"created","issue":{"url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/109","labels_url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/109/labels{/name}","comments_url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/109/comments","events_url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/109/events","html_url":"https://github.com/EmmittJ/PoESkillTree/pull/109","id":67504944,"number":109,"title":"Added a SkilltreeUndoState Class","user":{"login":"EmmittJ","id":7478125,"avatar_url":"https://avatars.githubusercontent.com/u/7478125?v=3","gravatar_id":"","url":"https://api.github.com/users/EmmittJ","html_url":"https://github.com/EmmittJ","followers_url":"https://api.github.com/users/EmmittJ/followers","following_url":"https://api.github.com/users/EmmittJ/following{/other_user}","gists_url":"https://api.github.com/users/EmmittJ/gists{/gist_id}","starred_url":"https://api.github.com/users/EmmittJ/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EmmittJ/subscriptions","organizations_url":"https://api.github.com/users/EmmittJ/orgs","repos_url":"https://api.github.com/users/EmmittJ/repos","events_url":"https://api.github.com/users/EmmittJ/events{/privacy}","received_events_url":"https://api.github.com/users/EmmittJ/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-04-10T04:09:51Z","updated_at":"2015-04-10T07:35:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/EmmittJ/PoESkillTree/pulls/109","html_url":"https://github.com/EmmittJ/PoESkillTree/pull/109","diff_url":"https://github.com/EmmittJ/PoESkillTree/pull/109.diff","patch_url":"https://github.com/EmmittJ/PoESkillTree/pull/109.patch"},"body":"I would like you guys to look over it and approve or not of it.\r\nThis is an update to a hack that I did for the undo feature.\r\nAs the title says I added a class which can be modified to hold anything and set anything we may need in the future. This is so we don't have to change undo logic, just the class. \r\n\r\nTell me what you think and if it is good enough to be pulled. "},"comment":{"url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/comments/91467960","html_url":"https://github.com/EmmittJ/PoESkillTree/pull/109#issuecomment-91467960","issue_url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/109","id":91467960,"user":{"login":"SpaceOgre","id":381657,"avatar_url":"https://avatars.githubusercontent.com/u/381657?v=3","gravatar_id":"","url":"https://api.github.com/users/SpaceOgre","html_url":"https://github.com/SpaceOgre","followers_url":"https://api.github.com/users/SpaceOgre/followers","following_url":"https://api.github.com/users/SpaceOgre/following{/other_user}","gists_url":"https://api.github.com/users/SpaceOgre/gists{/gist_id}","starred_url":"https://api.github.com/users/SpaceOgre/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SpaceOgre/subscriptions","organizations_url":"https://api.github.com/users/SpaceOgre/orgs","repos_url":"https://api.github.com/users/SpaceOgre/repos","events_url":"https://api.github.com/users/SpaceOgre/events{/privacy}","received_events_url":"https://api.github.com/users/SpaceOgre/received_events","type":"User","site_admin":false},"created_at":"2015-04-10T07:35:53Z","updated_at":"2015-04-10T07:35:53Z","body":"One thing: I'm not so sure that level should be part of the undo, since it does not affect the tree in any way. But I guess that depends on what we want undo to mean:\r\n* Only tree changes\r\n* All changes done to current build\r\n\r\nIf the first then I think we can remove the level thing and be happy. Otherwise more stuff needs to be added and more triggers for the undo stack needs to be added as well. Atm when you change level it will be added to the undo stack, it will only be added if you do a tree change as well. "}}