html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue https://github.com/ACEmulator/ACE/pull/243#issuecomment-294642122,https://api.github.com/repos/ACEmulator/ACE/issues/243,294642122,MDEyOklzc3VlQ29tbWVudDI5NDY0MjEyMg==,37453,2017-04-18T01:14:36Z,2017-04-18T01:14:36Z,CONTRIBUTOR,I've made changes to each of the commented sections - the code is now ready for another review.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,222023780 https://github.com/ACEmulator/ACE/pull/239#issuecomment-294742418,https://api.github.com/repos/ACEmulator/ACE/issues/239,294742418,MDEyOklzc3VlQ29tbWVudDI5NDc0MjQxOA==,26508216,2017-04-18T09:20:50Z,2017-04-18T09:20:50Z,CONTRIBUTOR,I removed the smite command as requested - although I don't see why implementing it was so bad if it's in the client.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,221953958 https://github.com/ACEmulator/ACE/pull/247#issuecomment-294856575,https://api.github.com/repos/ACEmulator/ACE/issues/247,294856575,MDEyOklzc3VlQ29tbWVudDI5NDg1NjU3NQ==,25460553,2017-04-18T14:09:58Z,2017-04-18T14:09:58Z,CONTRIBUTOR,I also really like the container class container -> creature -> player We will be able to do container -> chest. ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,222409472 https://github.com/ACEmulator/ACE/pull/236#issuecomment-294861667,https://api.github.com/repos/ACEmulator/ACE/issues/236,294861667,MDEyOklzc3VlQ29tbWVudDI5NDg2MTY2Nw==,37453,2017-04-18T14:26:25Z,2017-04-18T14:26:25Z,CONTRIBUTOR,Thanks for taking a look!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,221393225 https://github.com/ACEmulator/ACE/pull/251#issuecomment-294962639,https://api.github.com/repos/ACEmulator/ACE/issues/251,294962639,MDEyOklzc3VlQ29tbWVudDI5NDk2MjYzOQ==,26508216,2017-04-18T20:02:20Z,2017-04-18T20:02:20Z,CONTRIBUTOR,I like this structure. 👍 ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,222524134 https://github.com/ACEmulator/ACE/pull/251#issuecomment-294980138,https://api.github.com/repos/ACEmulator/ACE/issues/251,294980138,MDEyOklzc3VlQ29tbWVudDI5NDk4MDEzOA==,25351661,2017-04-18T20:56:44Z,2017-04-18T20:56:44Z,CONTRIBUTOR,"Folders should match namespaces, it's even a StyleCop rule. This type of object organization is not normal practice in industry and I'm not a fan. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,222524134 https://github.com/ACEmulator/ACE/pull/250#issuecomment-294980500,https://api.github.com/repos/ACEmulator/ACE/issues/250,294980500,MDEyOklzc3VlQ29tbWVudDI5NDk4MDUwMA==,25351661,2017-04-18T20:57:49Z,2017-04-18T20:57:49Z,CONTRIBUTOR,What's the use case for this? ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,222487087 https://github.com/ACEmulator/ACE/pull/250#issuecomment-295009469,https://api.github.com/repos/ACEmulator/ACE/issues/250,295009469,MDEyOklzc3VlQ29tbWVudDI5NTAwOTQ2OQ==,11369233,2017-04-18T22:59:12Z,2017-04-18T22:59:12Z,MEMBER,Use case is for house objects but per our discussion I should look further at this to refine to a more specific object,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,222487087