{"action":"created","issue":{"url":"https://api.github.com/repos/Civcraft/NameLayer/issues/212","repository_url":"https://api.github.com/repos/Civcraft/NameLayer","labels_url":"https://api.github.com/repos/Civcraft/NameLayer/issues/212/labels{/name}","comments_url":"https://api.github.com/repos/Civcraft/NameLayer/issues/212/comments","events_url":"https://api.github.com/repos/Civcraft/NameLayer/issues/212/events","html_url":"https://github.com/Civcraft/NameLayer/pull/212","id":169776826,"number":212,"title":"Async Changes","user":{"login":"suirad","id":8798829,"avatar_url":"https://avatars.githubusercontent.com/u/8798829?v=3","gravatar_id":"","url":"https://api.github.com/users/suirad","html_url":"https://github.com/suirad","followers_url":"https://api.github.com/users/suirad/followers","following_url":"https://api.github.com/users/suirad/following{/other_user}","gists_url":"https://api.github.com/users/suirad/gists{/gist_id}","starred_url":"https://api.github.com/users/suirad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/suirad/subscriptions","organizations_url":"https://api.github.com/users/suirad/orgs","repos_url":"https://api.github.com/users/suirad/repos","events_url":"https://api.github.com/users/suirad/events{/privacy}","received_events_url":"https://api.github.com/users/suirad/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-08-07T02:50:44Z","updated_at":"2016-08-12T00:30:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Civcraft/NameLayer/pulls/212","html_url":"https://github.com/Civcraft/NameLayer/pull/212","diff_url":"https://github.com/Civcraft/NameLayer/pull/212.diff","patch_url":"https://github.com/Civcraft/NameLayer/pull/212.patch"},"body":"- Adds new async methods for most db calls that return void.\r\n\r\n- Changes internal use of most(where appropriately applicable) db calls to use new async methods.\r\n\r\n- Added/changes mercury messaging to apply changes directly to cache, instead of forcing one(or many) db hits due to recaching.\r\n\r\n- Changed many group related methods to have alternate calls that take a boolean determining whether or not the call will modify the db, or only the local cache. Defaulting to true, preserving existing call-sites.\r\n\r\n- Standardized mercury access and plugin checks. Also localizing pretty much all mercury messaging to sites that trigger db events, essentially binding the two. So you can reliably expect that, if you look at places where the db is modified, you can expect a mercury message there, as the db change would only apply at the source server of the change. "},"comment":{"url":"https://api.github.com/repos/Civcraft/NameLayer/issues/comments/239332930","html_url":"https://github.com/Civcraft/NameLayer/pull/212#issuecomment-239332930","issue_url":"https://api.github.com/repos/Civcraft/NameLayer/issues/212","id":239332930,"user":{"login":"ProgrammerDan","id":409230,"avatar_url":"https://avatars.githubusercontent.com/u/409230?v=3","gravatar_id":"","url":"https://api.github.com/users/ProgrammerDan","html_url":"https://github.com/ProgrammerDan","followers_url":"https://api.github.com/users/ProgrammerDan/followers","following_url":"https://api.github.com/users/ProgrammerDan/following{/other_user}","gists_url":"https://api.github.com/users/ProgrammerDan/gists{/gist_id}","starred_url":"https://api.github.com/users/ProgrammerDan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ProgrammerDan/subscriptions","organizations_url":"https://api.github.com/users/ProgrammerDan/orgs","repos_url":"https://api.github.com/users/ProgrammerDan/repos","events_url":"https://api.github.com/users/ProgrammerDan/events{/privacy}","received_events_url":"https://api.github.com/users/ProgrammerDan/received_events","type":"User","site_admin":false},"created_at":"2016-08-12T00:30:30Z","updated_at":"2016-08-12T00:30:30Z","body":"Ok, I've reviewed. Sadly this PR is not ready for inclusion. Some of the issues are Namelayer's and not the fault of this Pull, and if it were those issues alone I would merge it now. However, some other issues do remain with this PR itself, and those need to be resolved or clarified before we can proceed."}}
{"actor":{"display_login":"ProgrammerDan"}}