{"html_url": "https://github.com/ACEmulator/ACE/pull/2#issuecomment-274909744", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/2", "id": 274909744, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NDkwOTc0NA==", "user": {"value": 6705011, "label": "xanxin887"}, "created_at": "2017-01-24T19:26:10Z", "updated_at": "2017-01-24T19:26:10Z", "author_association": "NONE", "body": "characters_database.sql that is included had a bad PK this one fixes that. \r\n\r\n\r\n[characters_database.sql.txt](https://github.com/ACEmulator/ACE/files/727740/characters_database.sql.txt)\r\n", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 202908658, "label": "Updates for DB 1/24/17"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/6#issuecomment-276565884", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/6", "id": 276565884, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NjU2NTg4NA==", "user": {"value": 25125434, "label": "TigrisOfGaul"}, "created_at": "2017-02-01T03:51:47Z", "updated_at": "2017-02-01T03:51:47Z", "author_association": "CONTRIBUTOR", "body": "Characters are saved to the database upon creation, just not fully. Appearance, stats, and skills still need to be persisted. I will try to finish this asap. ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204489997, "label": "Characters aren't saved or loaded properly yet from the DB"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/10#issuecomment-277009368", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/10", "id": 277009368, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzAwOTM2OA==", "user": {"value": 6111341, "label": "Rawaho"}, "created_at": "2017-02-02T16:36:57Z", "updated_at": "2017-02-02T16:36:57Z", "author_association": "CONTRIBUTOR", "body": "Additional checks will be added on account login as well.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204670023, "label": "Client and Server expects account in ace_auth to be lowercase only."}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/12#issuecomment-277143116", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/12", "id": 277143116, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzE0MzExNg==", "user": {"value": 25497527, "label": "firesickle"}, "created_at": "2017-02-03T01:52:42Z", "updated_at": "2017-02-03T01:52:42Z", "author_association": "NONE", "body": "As an alternative, You can run it in its current state with a scheduled task, which I think you can set up to execute upon start up. Additionally, you can set a scheduled task to run again if it is not already running, to get the client to start back up if it threw an exception and ended. Not sure myself what extra steps would need to make this into a service. In my past experience, I have had never-ending problems with custom built windows services that do web interactions, had a lot more success when switching to scheduled tasks", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205040357, "label": "Converting ACE to run as a Windows service"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/12#issuecomment-277153067", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/12", "id": 277153067, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzE1MzA2Nw==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-03T03:05:38Z", "updated_at": "2017-02-03T03:05:38Z", "author_association": "CONTRIBUTOR", "body": "Windows services are a real pain for development.  This won't need to be done for a really long time.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205040357, "label": "Converting ACE to run as a Windows service"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/11#issuecomment-277183931", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/11", "id": 277183931, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzE4MzkzMQ==", "user": {"value": 22650655, "label": "sbaum23"}, "created_at": "2017-02-03T07:40:42Z", "updated_at": "2017-02-03T07:40:42Z", "author_association": "CONTRIBUTOR", "body": "Added /teleto debug command to teleport to coordinates.  Z value is currently hard coded to 200 because it needs to be read from a file (dat?).  \r\n\r\nAlso, the two methods to convert Coordinates to XYZ are in DebugCommands.cs because I'm not sure where you want to put them. ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204803198, "label": "Run faster, jump higher, teleport to coordinates (rather than XYZ position)"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/11#issuecomment-277360874", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/11", "id": 277360874, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzM2MDg3NA==", "user": {"value": 22650655, "label": "sbaum23"}, "created_at": "2017-02-03T20:59:39Z", "updated_at": "2017-02-03T20:59:39Z", "author_association": "CONTRIBUTOR", "body": "I blew away my fork.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204803198, "label": "Run faster, jump higher, teleport to coordinates (rather than XYZ position)"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/13#issuecomment-277421543", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/13", "id": 277421543, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQyMTU0Mw==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-04T05:55:04Z", "updated_at": "2017-02-04T05:55:04Z", "author_association": "CONTRIBUTOR", "body": "Closing, new PR coming.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205077117, "label": "database layer encapsulation and abstraction"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/6#issuecomment-277444904", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/6", "id": 277444904, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ0NDkwNA==", "user": {"value": 25125434, "label": "TigrisOfGaul"}, "created_at": "2017-02-04T13:09:21Z", "updated_at": "2017-02-04T13:09:21Z", "author_association": "CONTRIBUTOR", "body": "Character persistence after creation is complete but work still needs to be done to load character stats, skills and appearance from the database for world entry. ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204489997, "label": "Characters aren't saved or loaded properly yet from the DB"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/20#issuecomment-277454303", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/20", "id": 277454303, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ1NDMwMw==", "user": {"value": 570040, "label": "zbuc"}, "created_at": "2017-02-04T15:48:45Z", "updated_at": "2017-02-04T15:48:45Z", "author_association": "CONTRIBUTOR", "body": "I suggest a reasonable first version of this might look like:\r\n\r\n1) an update to the core .sql file and `Config.json` to add a new `ace_global` database with a table `schema_revisions` tracking database schema versions\r\n\r\nThe columns might be something like:\r\n\r\n`id int(auto increment), schema_name varchar, schema_version int`\r\n\r\nalso insert initial values into the `schema_revisions` table to set the current `schema_version` at 0 on all databases (`ace_auth`, `ace_character`, `ace_world`).\r\n\r\n2) add a task that runs during server startup that uses the database names from `Config.json` and compares the .sql files found on the filesystem against the results from `ace_global.schema_revisions` and runs the new ones, and updates the database.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205353528, "label": "Server should have a way to apply DB migrations"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/21#issuecomment-277461220", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/21", "id": 277461220, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ2MTIyMA==", "user": {"value": 570040, "label": "zbuc"}, "created_at": "2017-02-04T17:29:13Z", "updated_at": "2017-02-04T17:29:13Z", "author_association": "CONTRIBUTOR", "body": "Fixes Issue #15 ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205361124, "label": "Change password hashing to use scrypt"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/18#issuecomment-277475417", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/18", "id": 277475417, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ3NTQxNw==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-04T20:34:39Z", "updated_at": "2017-02-04T20:34:39Z", "author_association": "MEMBER", "body": "In looking at the character deletion and restore section it made sense to me that name in ace_character shouldn't be made a unique value\r\nAt present in master, even if that character was marked as deleted, the character name is held.\r\n\r\nI've made changes to the code and database to better emulate how it worked before with respect to deletions and restores.\r\n\r\nAs is, no character is ever marked deleted so that is yet to be implemented, although I suspect that would be a server side SQL script or command within ACE running at an interval/manually via console as determined by a configuration variable(s) possibly in config.json for how often it checks, how long the restore window is and if to mark characters deleted or not which would remove them from restoration via client. \r\n\r\nGoing further than original design, perhaps it could be changed to hold the character name for that account only while the restoration window is active, as well as only allowing that account to create a brand new character with that deleted name. ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205330194, "label": "Option to Enable Automatic Account Creation"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/18#issuecomment-277481983", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/18", "id": 277481983, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ4MTk4Mw==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-04T22:14:16Z", "updated_at": "2017-02-04T22:14:16Z", "author_association": "MEMBER", "body": "Closing this PR as I have split up the several changes I made into separate PRs per request.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205330194, "label": "Option to Enable Automatic Account Creation"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/23#issuecomment-277483519", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/23", "id": 277483519, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ4MzUxOQ==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-04T22:42:06Z", "updated_at": "2017-02-04T22:42:06Z", "author_association": "MEMBER", "body": "Makes a lot of sense to me. Resetting the current config.json for each branch/pull is a bit of a hassle currently, this way you only have to reset when the file NEEDS to change", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205377812, "label": "Add Config.json to .gitignore"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/20#issuecomment-277485102", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/20", "id": 277485102, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ4NTEwMg==", "user": {"value": 570040, "label": "zbuc"}, "created_at": "2017-02-04T23:12:57Z", "updated_at": "2017-02-04T23:12:57Z", "author_association": "CONTRIBUTOR", "body": "Almost done with this...", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205353528, "label": "Server should have a way to apply DB migrations"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/8#issuecomment-277488613", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/8", "id": 277488613, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ4ODYxMw==", "user": {"value": 25472382, "label": "Oykib"}, "created_at": "2017-02-05T00:35:02Z", "updated_at": "2017-02-05T00:35:02Z", "author_association": "NONE", "body": "[ACEmuLauncher.zip](https://github.com/ACEmulator/ACE/files/752670/ACEmuLauncher.zip)\r\n", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204490371, "label": "Client Launcher needed"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/14#issuecomment-277492721", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/14", "id": 277492721, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ5MjcyMQ==", "user": {"value": 22650655, "label": "sbaum23"}, "created_at": "2017-02-05T02:21:27Z", "updated_at": "2017-02-05T02:21:27Z", "author_association": "CONTRIBUTOR", "body": "I wasn't following coding conventions.  Fixed that on a new branch and submitted a new PR.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205270553, "label": "Added /teleto command"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/28#issuecomment-277496824", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/28", "id": 277496824, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ5NjgyNA==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-05T04:28:54Z", "updated_at": "2017-02-05T04:28:54Z", "author_association": "MEMBER", "body": "Ignore this issue.. I was not fully understanding how the IPs were being used in config.json.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205389442, "label": "Change way mysql databases are accessed"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/31#issuecomment-277497148", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/31", "id": 277497148, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ5NzE0OA==", "user": {"value": 570040, "label": "zbuc"}, "created_at": "2017-02-05T04:39:10Z", "updated_at": "2017-02-05T04:39:10Z", "author_association": "CONTRIBUTOR", "body": "I think this would involve:\r\n\r\n1) Add the log path and verbosity flags to the `Config.json`\r\n2) Create a new persistent logging thread that can operate off a queue of log message tasks, sending them to the filesystem\r\n3) Create a new window to display log messages, streamed from the filesystem\r\n4) Replace `Console.WriteLine` messages used for logging with calls to queue log message tasks on the logging thread\r\n\r\nI'm not a .NET developer, but it looks like these classes might be useful: https://msdn.microsoft.com/en-us/library/microsoft.visualbasic.logging.filelogtracelistener.aspx", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205400067, "label": "Logs should go to separate window, and filesystem"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/30#issuecomment-277498912", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/30", "id": 277498912, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ5ODkxMg==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-05T05:37:01Z", "updated_at": "2017-02-05T05:37:01Z", "author_association": "MEMBER", "body": "Did the merge wrong again, apologies.. sending new PR shortly.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205398529, "label": "Fix for NAT issues on internal connections to WorldServer"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/29#issuecomment-277499579", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/29", "id": 277499579, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzQ5OTU3OQ==", "user": {"value": 22650655, "label": "sbaum23"}, "created_at": "2017-02-05T06:01:09Z", "updated_at": "2017-02-05T06:01:09Z", "author_association": "CONTRIBUTOR", "body": "Going to do a clean PR.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205394658, "label": "Teleto command redo"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/25#issuecomment-277505687", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/25", "id": 277505687, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzUwNTY4Nw==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-05T08:52:53Z", "updated_at": "2017-02-05T08:52:53Z", "author_association": "MEMBER", "body": "See PR #34 ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205381602, "label": "Option to Enable Automatic Account Creation"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/26#issuecomment-277506132", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/26", "id": 277506132, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzUwNjEzMg==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-05T09:03:41Z", "updated_at": "2017-02-05T09:03:41Z", "author_association": "MEMBER", "body": "Closing this due to conflicts, see PR #35", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205381707, "label": "Fixing up Character Delete and Restore functions"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/36#issuecomment-277545133", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/36", "id": 277545133, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzU0NTEzMw==", "user": {"value": 570040, "label": "zbuc"}, "created_at": "2017-02-05T20:06:01Z", "updated_at": "2017-02-05T20:06:01Z", "author_association": "CONTRIBUTOR", "body": "I'm not a C# guy and there are some stylistic things I don't like, mainly calling out to `DatabaseManager.Global` in the main `Database` class is very hacky, as is what I had to do with the `Global` prepared statements.\r\n\r\nBut, it all works and I think is good for a first pass.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205449659, "label": "Add automatic database migrations to startup"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/32#issuecomment-277549255", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/32", "id": 277549255, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzU0OTI1NQ==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-05T21:05:00Z", "updated_at": "2017-02-05T21:05:00Z", "author_association": "MEMBER", "body": "See PR #37 instead", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205403112, "label": "Fix for NAT issues on internal connections to WorldServer"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/36#issuecomment-277551247", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/36", "id": 277551247, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzU1MTI0Nw==", "user": {"value": 570040, "label": "zbuc"}, "created_at": "2017-02-05T21:32:29Z", "updated_at": "2017-02-05T21:32:29Z", "author_association": "CONTRIBUTOR", "body": "Fixes #20 ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205449659, "label": "Add automatic database migrations to startup"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/8#issuecomment-277572855", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/8", "id": 277572855, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzU3Mjg1NQ==", "user": {"value": 25472382, "label": "Oykib"}, "created_at": "2017-02-06T02:24:53Z", "updated_at": "2017-02-06T02:24:53Z", "author_association": "NONE", "body": "[ACEmuLauncher v0.0.0.3 source.zip](https://github.com/ACEmulator/ACE/files/753512/ACEmuLauncher.v0.0.0.3.source.zip)\r\nLatest version that has profiles.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204490371, "label": "Client Launcher needed"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/42#issuecomment-277735330", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/42", "id": 277735330, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzczNTMzMA==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-06T16:30:40Z", "updated_at": "2017-02-06T16:30:40Z", "author_association": "CONTRIBUTOR", "body": "This is low priority.  Wanted it logged as legit to-do, though.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205637204, "label": "Subscription data model concept + GLS replacement"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/5#issuecomment-277744701", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/5", "id": 277744701, "node_id": "MDEyOklzc3VlQ29tbWVudDI3Nzc0NDcwMQ==", "user": {"value": 25387933, "label": "Forbiddenz"}, "created_at": "2017-02-06T17:01:51Z", "updated_at": "2017-02-06T17:01:51Z", "author_association": "CONTRIBUTOR", "body": "Working on it, but going to need a landblock system I think before we can get very far on this..", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204489878, "label": "ACE Chat System"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/44#issuecomment-277752941", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/44", "id": 277752941, "node_id": "MDEyOklzc3VlQ29tbWVudDI3Nzc1Mjk0MQ==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-06T17:30:14Z", "updated_at": "2017-02-06T17:30:14Z", "author_association": "CONTRIBUTOR", "body": "Kaos: We just need a simple rank system and /setrank command or similar that flows through. 1=registered, 2=envoy, 3=sentinel, 4=admin, etc... Everyone goes under 1 automatically and an admin can use the /setrank command to give out any additional ranks. Value should be stored in DB.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205646390, "label": "Add column to authenticationDB.account for access rights"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/44#issuecomment-277754381", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/44", "id": 277754381, "node_id": "MDEyOklzc3VlQ29tbWVudDI3Nzc1NDM4MQ==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-06T17:35:16Z", "updated_at": "2017-02-06T17:35:16Z", "author_association": "MEMBER", "body": "for reference, values assigned to characters:\r\n\r\nSetPropertyBool(PropertyBool.IsPsr, false);\r\nSetPropertyBool(PropertyBool.IsAdmin, true);\r\nSetPropertyBool(PropertyBool.IsArch, false);\r\nSetPropertyBool(PropertyBool.IsSentinel, false);\r\nSetPropertyBool(PropertyBool.IsAdvocate, false);\r\n\r\nwould think it goes like this:\r\n\r\n0=player\r\n1=advocate (isAdvocate)\r\n2=sentinel (isSentinel)\r\n3=envoy (isPSR)\r\n4=developer? (IsArch)\r\n5=admin (isAdmin)", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205646390, "label": "Add column to authenticationDB.account for access rights"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/5#issuecomment-277760878", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/5", "id": 277760878, "node_id": "MDEyOklzc3VlQ29tbWVudDI3Nzc2MDg3OA==", "user": {"value": 8607407, "label": "kaos00723"}, "created_at": "2017-02-06T17:57:58Z", "updated_at": "2017-02-06T17:57:58Z", "author_association": "CONTRIBUTOR", "body": "From Rawaho: \"The chat really needs broadcast support and to do that we need a working grid system and to do that we need to load the dat files\r\nI am working on handling the dat files but haven't had much time the last few days\"", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204489878, "label": "ACE Chat System"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/6#issuecomment-277778434", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/6", "id": 277778434, "node_id": "MDEyOklzc3VlQ29tbWVudDI3Nzc3ODQzNA==", "user": {"value": 25125434, "label": "TigrisOfGaul"}, "created_at": "2017-02-06T19:00:52Z", "updated_at": "2017-02-06T19:00:52Z", "author_association": "CONTRIBUTOR", "body": "Mogwai is working on loading character data from the db.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204489997, "label": "Characters aren't saved or loaded properly yet from the DB"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/43#issuecomment-277875015", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/43", "id": 277875015, "node_id": "MDEyOklzc3VlQ29tbWVudDI3Nzg3NTAxNQ==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-07T01:43:41Z", "updated_at": "2017-02-07T01:43:41Z", "author_association": "CONTRIBUTOR", "body": "In progress.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205645250, "label": "Move Database Access and object models to a separate C# project"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/43#issuecomment-277889298", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/43", "id": 277889298, "node_id": "MDEyOklzc3VlQ29tbWVudDI3Nzg4OTI5OA==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-07T03:13:54Z", "updated_at": "2017-02-07T03:13:54Z", "author_association": "CONTRIBUTOR", "body": "As I'm working on this, it quickly turned into a major mess.  In order to extract this stuff I need to extract serialization and configuration from the data tier.  I couldn't do this to my satisfaction in 2 hours tonight and really need to talk to Rawaho about it before continuing.  Back burnering for now.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205645250, "label": "Move Database Access and object models to a separate C# project"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/6#issuecomment-277907609", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/6", "id": 277907609, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzkwNzYwOQ==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-07T05:46:32Z", "updated_at": "2017-02-07T06:39:31Z", "author_association": "CONTRIBUTOR", "body": "#47 has the rest of this issue done.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204489997, "label": "Characters aren't saved or loaded properly yet from the DB"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/44#issuecomment-277907896", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/44", "id": 277907896, "node_id": "MDEyOklzc3VlQ29tbWVudDI3NzkwNzg5Ng==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-07T05:48:38Z", "updated_at": "2017-02-07T05:48:38Z", "author_association": "CONTRIBUTOR", "body": "Had a discussion about this in Discord tonight.  The client sends 2 uints for IsEnvoy and IsAdmin.  Need to clarify if this is 1 field with a bitmask, 1 field with tiered elevation, or multiple fields.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205646390, "label": "Add column to authenticationDB.account for access rights"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/31#issuecomment-277995357", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/31", "id": 277995357, "node_id": "MDEyOklzc3VlQ29tbWVudDI3Nzk5NTM1Nw==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-07T13:11:47Z", "updated_at": "2017-02-07T13:11:47Z", "author_association": "CONTRIBUTOR", "body": ".Net guy recommendation: use NLog or Log4Net.  both are well known, reliable frameworks for logging.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205400067, "label": "Logs should go to separate window, and filesystem"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/47#issuecomment-278051575", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/47", "id": 278051575, "node_id": "MDEyOklzc3VlQ29tbWVudDI3ODA1MTU3NQ==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-07T16:22:46Z", "updated_at": "2017-02-07T16:22:46Z", "author_association": "CONTRIBUTOR", "body": "Closing PR: sql script does not specify unsigned types", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205792049, "label": "Load saved characters"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/49#issuecomment-278546199", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/49", "id": 278546199, "node_id": "MDEyOklzc3VlQ29tbWVudDI3ODU0NjE5OQ==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-09T04:46:14Z", "updated_at": "2017-02-09T04:46:14Z", "author_association": "CONTRIBUTOR", "body": "in PR #50 ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205882786, "label": "XP expenditure on skills/stats"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/48#issuecomment-278546228", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/48", "id": 278546228, "node_id": "MDEyOklzc3VlQ29tbWVudDI3ODU0NjIyOA==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-09T04:46:26Z", "updated_at": "2017-02-09T04:46:26Z", "author_association": "CONTRIBUTOR", "body": "in PR #50 ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205882576, "label": "Add all other properties to Character persistence model"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/51#issuecomment-278870195", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/51", "id": 278870195, "node_id": "MDEyOklzc3VlQ29tbWVudDI3ODg3MDE5NQ==", "user": {"value": 6111341, "label": "Rawaho"}, "created_at": "2017-02-10T06:44:43Z", "updated_at": "2017-02-10T06:44:43Z", "author_association": "CONTRIBUTOR", "body": "Should be a combination of the first 2, we can't just have only on logout, if a player is in a long play session and the server crashes for whatever reason it would be a massive rollback.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 206403228, "label": "Saving a Character to the DB"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/35#issuecomment-279123506", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/35", "id": 279123506, "node_id": "MDEyOklzc3VlQ29tbWVudDI3OTEyMzUwNg==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-11T05:46:23Z", "updated_at": "2017-02-11T05:46:23Z", "author_association": "MEMBER", "body": "See PR #57 ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205412246, "label": "Fixing up Character Delete and Restore functions"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/55#issuecomment-279125244", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/55", "id": 279125244, "node_id": "MDEyOklzc3VlQ29tbWVudDI3OTEyNTI0NA==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-11T06:27:53Z", "updated_at": "2017-02-11T06:27:53Z", "author_association": "MEMBER", "body": "resolved merge conflict", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 206410568, "label": "Stubbed CommandHandlers for known AdminCommands"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/51#issuecomment-279128317", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/51", "id": 279128317, "node_id": "MDEyOklzc3VlQ29tbWVudDI3OTEyODMxNw==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-11T07:49:45Z", "updated_at": "2017-02-11T07:49:45Z", "author_association": "MEMBER", "body": "Old school AC showed a message each time a character was saved.\r\n\r\n\"[charactername] has been saved.\"\r\n\r\nit was probably on a 5 or 10 minute interval. They later just hid the message.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 206403228, "label": "Saving a Character to the DB"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/36#issuecomment-279195536", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/36", "id": 279195536, "node_id": "MDEyOklzc3VlQ29tbWVudDI3OTE5NTUzNg==", "user": {"value": 570040, "label": "zbuc"}, "created_at": "2017-02-12T04:10:14Z", "updated_at": "2017-02-12T04:10:14Z", "author_association": "CONTRIBUTOR", "body": "I cleaned up and simplified some of the code, and rebased against `master`.\r\n\r\nAll current migrations run and the database is in a working state after compiling and running off this branch.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205449659, "label": "Add automatic database migrations to startup"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/57#issuecomment-279196637", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/57", "id": 279196637, "node_id": "MDEyOklzc3VlQ29tbWVudDI3OTE5NjYzNw==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-12T04:44:40Z", "updated_at": "2017-02-12T04:49:15Z", "author_association": "MEMBER", "body": "Latest commit finishes up the character deletion function and replicates how deletions occurred on the retail worlds. To my knowledge, characters were never truly deleted beyond this stage and were always left in the database to be restored via /bornagain if warranted.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 206960724, "label": "Fixing up Character Delete and Restore functions"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/36#issuecomment-279226754", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/36", "id": 279226754, "node_id": "MDEyOklzc3VlQ29tbWVudDI3OTIyNjc1NA==", "user": {"value": 25387933, "label": "Forbiddenz"}, "created_at": "2017-02-12T15:43:24Z", "updated_at": "2017-02-12T15:43:24Z", "author_association": "CONTRIBUTOR", "body": "This is s great idea but I do wonder if this entire process should be separated into its own project in the solution.  I.e. Patcher because this is not really a server process but more so a patcher / installer issue", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205449659, "label": "Add automatic database migrations to startup"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/7#issuecomment-279227233", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/7", "id": 279227233, "node_id": "MDEyOklzc3VlQ29tbWVudDI3OTIyNzIzMw==", "user": {"value": 25387933, "label": "Forbiddenz"}, "created_at": "2017-02-12T15:50:35Z", "updated_at": "2017-02-12T15:50:35Z", "author_association": "CONTRIBUTOR", "body": "It's not just world object movement but it's the entire world object.  It needs implemented correctly before any major feature can be added or else the rework will be awful.  The current world object needs uncoupled and it should be broken down into multiple sub objects to handle the correct sub object layers like sub models etc .  If dive right it could be combined with database tables to generate objects correctly based on sql table data ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204490022, "label": "World object movement / animation needs proper implementing"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/51#issuecomment-279281301", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/51", "id": 279281301, "node_id": "MDEyOklzc3VlQ29tbWVudDI3OTI4MTMwMQ==", "user": {"value": 20557688, "label": "spikeone83"}, "created_at": "2017-02-13T02:39:40Z", "updated_at": "2017-02-13T02:39:40Z", "author_association": "NONE", "body": "best is to save characters routinely, characters not saving often server side is one of the easy ways of duping in any online game", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 206403228, "label": "Saving a Character to the DB"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/8#issuecomment-279350491", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/8", "id": 279350491, "node_id": "MDEyOklzc3VlQ29tbWVudDI3OTM1MDQ5MQ==", "user": {"value": 4389226, "label": "damnubber"}, "created_at": "2017-02-13T10:37:53Z", "updated_at": "2017-02-13T10:37:53Z", "author_association": "NONE", "body": "Any specific reason this isn't in the ACE solution? ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204490371, "label": "Client Launcher needed"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/8#issuecomment-279395678", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/8", "id": 279395678, "node_id": "MDEyOklzc3VlQ29tbWVudDI3OTM5NTY3OA==", "user": {"value": 25472382, "label": "Oykib"}, "created_at": "2017-02-13T13:43:20Z", "updated_at": "2017-02-13T13:43:20Z", "author_association": "NONE", "body": "This can be deleted. I was informed that they do not need a launcher developed and have one in mind already. ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204490371, "label": "Client Launcher needed"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/31#issuecomment-280018556", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/31", "id": 280018556, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDAxODU1Ng==", "user": {"value": 12808515, "label": "Zegeger"}, "created_at": "2017-02-15T14:03:38Z", "updated_at": "2017-02-15T14:03:38Z", "author_association": "CONTRIBUTOR", "body": "I think ideally it should be written such that different logging listeners could be swapped in allowing a logger of ones choice.\r\nMy recommendation would be to do ETW logging.  Its more cumbersome to handle the resulting logs, but there is a very low perf overhead, allowing you to hit 10k+ entries per second without stealing perf.  This is what things like Exchange/Skype for Business servers use.\r\nI could work on something like this.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205400067, "label": "Logs should go to separate window, and filesystem"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/57#issuecomment-280233728", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/57", "id": 280233728, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDIzMzcyOA==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-16T05:17:24Z", "updated_at": "2017-02-16T05:17:24Z", "author_association": "MEMBER", "body": "See PR #66 for fixes.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 206960724, "label": "Fixing up Character Delete and Restore functions"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/20#issuecomment-280469751", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/20", "id": 280469751, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDQ2OTc1MQ==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-16T21:39:47Z", "updated_at": "2017-02-16T21:39:47Z", "author_association": "CONTRIBUTOR", "body": "this was recently done with a powershell script that was merged.  if you're ok with using that, please close the issue.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205353528, "label": "Server should have a way to apply DB migrations"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/8#issuecomment-280472468", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/8", "id": 280472468, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDQ3MjQ2OA==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-16T21:49:40Z", "updated_at": "2017-02-16T21:49:40Z", "author_association": "CONTRIBUTOR", "body": "Thwargle has one, hoping to release it today.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204490371, "label": "Client Launcher needed"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/4#issuecomment-280479110", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/4", "id": 280479110, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDQ3OTExMA==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-16T22:13:18Z", "updated_at": "2017-02-16T22:13:18Z", "author_association": "CONTRIBUTOR", "body": "Elaborate?", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204489832, "label": "ACE World DB Design"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/55#issuecomment-280565958", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/55", "id": 280565958, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDU2NTk1OA==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-17T06:30:40Z", "updated_at": "2017-02-17T06:30:40Z", "author_association": "MEMBER", "body": "See #67", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 206410568, "label": "Stubbed CommandHandlers for known AdminCommands"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/54#issuecomment-280581225", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/54", "id": 280581225, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDU4MTIyNQ==", "user": {"value": 22650655, "label": "sbaum23"}, "created_at": "2017-02-17T08:00:32Z", "updated_at": "2017-02-17T08:00:32Z", "author_association": "CONTRIBUTOR", "body": "Closing this PR because so much stuff had changed since this was submitted.  Just opened a new PR.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 206407101, "label": "Added friends list functionality."}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/21#issuecomment-280604951", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/21", "id": 280604951, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDYwNDk1MQ==", "user": {"value": 10608427, "label": "Mag-nus"}, "created_at": "2017-02-17T09:47:21Z", "updated_at": "2017-02-17T09:47:21Z", "author_association": "MEMBER", "body": "I like the idea, but, i don't feel it's necessary to visit until we know how account create will actually work.\r\n\r\nMerging it into current though seems fine. Probably better sooner rather than later as I assume accounts will have to be recreated.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205361124, "label": "Change password hashing to use scrypt"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/36#issuecomment-280605128", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/36", "id": 280605128, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDYwNTEyOA==", "user": {"value": 10608427, "label": "Mag-nus"}, "created_at": "2017-02-17T09:48:15Z", "updated_at": "2017-02-17T09:48:15Z", "author_association": "MEMBER", "body": "I'm not fond of the complexity for this. What i'd like to see instead is us merge our updates down into the base sql file at some point, and, in the future, when new updates are added, also update the base sql file as well, so a new install wouldn't need to apply the updates, as they'd be.. up to date. (i'm by no means a db person so i don't know if this is against the norm)", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205449659, "label": "Add automatic database migrations to startup"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/37#issuecomment-280605245", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/37", "id": 280605245, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDYwNTI0NQ==", "user": {"value": 10608427, "label": "Mag-nus"}, "created_at": "2017-02-17T09:48:46Z", "updated_at": "2017-02-17T09:48:46Z", "author_association": "MEMBER", "body": "rawaho made a change that might have fixed the issue you were trying to fix.\r\n\r\nI think IP binding should be explicit. We might get a case where a user has 2 ip addresses on the same box and choses to bind each ip to a unique world, using the same port.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205454231, "label": "Add option to send different host address than external host address during world handoff"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/36#issuecomment-280639466", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/36", "id": 280639466, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDYzOTQ2Ng==", "user": {"value": 25387933, "label": "Forbiddenz"}, "created_at": "2017-02-17T12:43:40Z", "updated_at": "2017-02-17T12:43:40Z", "author_association": "CONTRIBUTOR", "body": "Hi Mag-nus,\r\nThe problem with your idea is some asshat will figure out how to screw up their original db.\r\n\r\nI worked on many production level database patches and they pretty much all start at step x and move forward where x can be the current software version.\r\n\r\nYour right as a new installer needs to run from version 0 and then if version 50 removes a table in version 0 it will be a useless step and take more time.\r\n\r\nBut the benefits of the system far outweigh the useless garbage.\r\n\r\nWith a design that goes by version and up you can guarantee that the database is always standard and it eliminates the room for error with things getting out of sync or even missed.\r\n\r\nWhen you have the step process it's a little more work and sometimes seems dumb but it keeps all the db updates on a forward track even if they seem to be doing things the slow way.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205449659, "label": "Add automatic database migrations to startup"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/37#issuecomment-280717424", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/37", "id": 280717424, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDcxNzQyNA==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-17T17:42:37Z", "updated_at": "2017-02-17T17:42:37Z", "author_association": "MEMBER", "body": "it makes sense to me add variables for each ip address we explicitly send. It wouldn't really fix what this PR intention was which is to basically let me set up an external ip for external clients, and an internal ip for internal clients. I understand if its not something we want to put in to the master code, I'll just end up keeping a branch off my fork to roll in each time i update my server.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205454231, "label": "Add option to send different host address than external host address during world handoff"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/73#issuecomment-280844895", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/73", "id": 280844895, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDg0NDg5NQ==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-18T13:14:51Z", "updated_at": "2017-02-18T13:14:51Z", "author_association": "CONTRIBUTOR", "body": "I like it, but you need to resolve the merge conflicts first.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 208616825, "label": "Network Refactor"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/73#issuecomment-280846918", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/73", "id": 280846918, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDg0NjkxOA==", "user": {"value": 10608427, "label": "Mag-nus"}, "created_at": "2017-02-18T13:53:06Z", "updated_at": "2017-02-18T13:53:06Z", "author_association": "MEMBER", "body": "Also, double check your namespaces. It looks like GameMessageOpcode isn't in the correct namespace. Namespaces should match the folder depth of the file.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 208616825, "label": "Network Refactor"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/45#issuecomment-280874242", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/45", "id": 280874242, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDg3NDI0Mg==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-18T20:54:14Z", "updated_at": "2017-02-18T20:54:14Z", "author_association": "MEMBER", "body": "completed known admin commands. not sure if we need to stub client commands just to protect them from being used as the client handles them already", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205647107, "label": "Stub CommandHandlers for in-game commands"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/58#issuecomment-280902049", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/58", "id": 280902049, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDkwMjA0OQ==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-19T07:36:04Z", "updated_at": "2017-02-19T07:36:04Z", "author_association": "MEMBER", "body": "Ignore this PR, will close shortly.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 207015535, "label": "Added support on ACE console for setting per character admin rights and character renaming"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/9#issuecomment-280936571", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/9", "id": 280936571, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDkzNjU3MQ==", "user": {"value": 12808515, "label": "Zegeger"}, "created_at": "2017-02-19T18:09:45Z", "updated_at": "2017-02-19T18:09:45Z", "author_association": "CONTRIBUTOR", "body": "You can assign this to me", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204506877, "label": "Rework packet pipe"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/77#issuecomment-280954420", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/77", "id": 280954420, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDk1NDQyMA==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-19T22:20:47Z", "updated_at": "2017-02-20T01:39:40Z", "author_association": "MEMBER", "body": "Added functionality for /sudo as described above.\r\n\r\nexample: sudo othercommand othercommandsparamters\r\n\r\nsudo set-characteraccess charactername admin\r\n\r\nsudo telepoi arwic\r\n\r\nthe above examples would work on characters not set as admins as long as the account you're logged in as is set with admin privileges.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 208701617, "label": "Add AccessLevel to Accounts"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/77#issuecomment-280956235", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/77", "id": 280956235, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDk1NjIzNQ==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-19T22:46:04Z", "updated_at": "2017-02-19T22:46:04Z", "author_association": "MEMBER", "body": "Adjusted the set commands to work on both the console and in-game. The first account has to be created via console and also set as an admin for these to work in game.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 208701617, "label": "Add AccessLevel to Accounts"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/5#issuecomment-280959920", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/5", "id": 280959920, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDk1OTkyMA==", "user": {"value": 10608427, "label": "Mag-nus"}, "created_at": "2017-02-19T23:44:32Z", "updated_at": "2017-02-19T23:44:32Z", "author_association": "MEMBER", "body": "Local (say), and tell now work.\r\nFix: The names are not clickable. Not sure why.\r\nFix: Local (say) is world global for now. It needs a range restriction of some sort.\r\n\r\nNeed to add joining and using of the chat channels (general, trade, etc...)", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204489878, "label": "ACE Chat System"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/76#issuecomment-280978914", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/76", "id": 280978914, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDk3ODkxNA==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-20T03:19:57Z", "updated_at": "2017-02-20T03:19:57Z", "author_association": "CONTRIBUTOR", "body": "Resolve the conflicts and we'll approve as is.  We'll deal with persistence later.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 208656503, "label": "Reworked Appear Offline functionality"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/77#issuecomment-280989331", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/77", "id": 280989331, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MDk4OTMzMQ==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-20T05:08:23Z", "updated_at": "2017-02-20T05:08:23Z", "author_association": "MEMBER", "body": "@fantoms rightly pointed out that the most recent master changes changed SendSystemMessage to SendServerMessage with an enum argument for message type. Corrected that issue as well.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 208701617, "label": "Add AccessLevel to Accounts"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/9#issuecomment-281144852", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/9", "id": 281144852, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MTE0NDg1Mg==", "user": {"value": 8607407, "label": "kaos00723"}, "created_at": "2017-02-20T17:58:04Z", "updated_at": "2017-02-20T17:58:04Z", "author_association": "CONTRIBUTOR", "body": "@Zegeger , issues can only be assigned to the issue starter or a member of the org. You're the only person working on this currently so when you submit the PR just reference issue #9 and it will close automatically after being merged.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204506877, "label": "Rework packet pipe"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/20#issuecomment-281223009", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/20", "id": 281223009, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MTIyMzAwOQ==", "user": {"value": 570040, "label": "zbuc"}, "created_at": "2017-02-21T01:51:53Z", "updated_at": "2017-02-21T01:51:53Z", "author_association": "CONTRIBUTOR", "body": "The script can only recreate the initial database table state completely so it won't be useful for server operators.\r\n\r\nIf it were extended with a second script solely for running upgrades that could keep track of the currently run database upgrades somewhere, for example in a flat-file in a temporary directory, so they wouldn't be run again that would be useful.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205353528, "label": "Server should have a way to apply DB migrations"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/9#issuecomment-281332628", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/9", "id": 281332628, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MTMzMjYyOA==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-21T12:34:45Z", "updated_at": "2017-02-21T12:34:45Z", "author_association": "CONTRIBUTOR", "body": "Zegeger's work is now merged.  Closing the issue.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204506877, "label": "Rework packet pipe"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/20#issuecomment-281333010", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/20", "id": 281333010, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MTMzMzAxMA==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-21T12:36:41Z", "updated_at": "2017-02-21T12:36:41Z", "author_association": "CONTRIBUTOR", "body": "The scripts would need to be altered to be re-run safe.  MySQL's support for this is spotty, but it could be done.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205353528, "label": "Server should have a way to apply DB migrations"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/20#issuecomment-281344829", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/20", "id": 281344829, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MTM0NDgyOQ==", "user": {"value": 570040, "label": "zbuc"}, "created_at": "2017-02-21T13:31:44Z", "updated_at": "2017-02-21T13:31:44Z", "author_association": "CONTRIBUTOR", "body": "For a solution that doesn't require migration authors to worry about that, do something similar to the pull request I created where it tracks the database migrations that have run and only runs them once, in sequential order. \n\nThat is relatively standard for database-backed applications.\n\nYou will also run into difficulty with migrations where they are interdependent, i.e. one migration needs to run before another one can be run, because it creates or alters tables used by the other migration. \n\nMy pull request took all this into account. \n\nWith your PowerShell solution you could probably track the migrations that have run in a text file or registry key.\n\n> On Feb 21, 2017, at 7:36 AM, Mogwai-AC <notifications@github.com> wrote:\n> \n> The scripts would need to be altered to be re-run safe. MySQL's support for this is spotty, but it could be done.\n> \n> \u2014\n> You are receiving this because you authored the thread.\n> Reply to this email directly, view it on GitHub, or mute the thread.\n> \n", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205353528, "label": "Server should have a way to apply DB migrations"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/84#issuecomment-282458956", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/84", "id": 282458956, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MjQ1ODk1Ng==", "user": {"value": 37453, "label": "fantoms"}, "created_at": "2017-02-25T04:23:07Z", "updated_at": "2017-02-25T04:23:07Z", "author_association": "CONTRIBUTOR", "body": "I was able to get the skills to lock into Infinity!, but the Abilities do not continue to spend after clicking 12 times on the +10 button; in other words - once you spend more then 121 points on abilities specificially, you can't spend anymore on abilities.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 209354215, "label": "Skill spend upper bound check too prevent xp crash, also Max Skills!"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/5#issuecomment-282489796", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/5", "id": 282489796, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MjQ4OTc5Ng==", "user": {"value": 10608427, "label": "Mag-nus"}, "created_at": "2017-02-25T15:09:21Z", "updated_at": "2017-02-25T15:09:21Z", "author_association": "MEMBER", "body": "Fix: Local chat still broadcasts globally\r\nTest/Fix: Chat Channels (General, LFG, ...) are broadcast to everyone. We should probably check if the recipient subscribes to the channel first before sending the message to them.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 204489878, "label": "ACE Chat System"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/90#issuecomment-282491222", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/90", "id": 282491222, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MjQ5MTIyMg==", "user": {"value": 10608427, "label": "Mag-nus"}, "created_at": "2017-02-25T15:32:31Z", "updated_at": "2017-02-25T15:32:31Z", "author_association": "MEMBER", "body": "GameMessageF7E5 should be named GameMessageDDD_Interrogation\r\n\r\nThere's a lot of hard coded group types still: 0x5, 0x9, 0xA, etc.. We should use an enum for the groups.\r\n\r\nNetworkSession.cs, and anywhere else, should use DateTime.UtcNow instead of DateTime.Now\r\n\r\nThis is a big PR so it will require further review.. I like what I see so far though, can't wait to get this merged.. I'll have to look at it a bit more later, and will want Mogwai to look at it as well.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 210217148, "label": "Rewrote network stack"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/90#issuecomment-282501819", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/90", "id": 282501819, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MjUwMTgxOQ==", "user": {"value": 10608427, "label": "Mag-nus"}, "created_at": "2017-02-25T18:14:35Z", "updated_at": "2017-02-25T18:14:35Z", "author_association": "MEMBER", "body": "Another thing I'm noticing is that if you connect, close the client with the X when you're in portal space, and reconnect again.. repeat this a few times, the client will eventually crash, and the server seems to stop listening to new connections. \r\n\r\nEven after the server forcibly removes the clients, new connections are still not being accepted.\r\n\r\nhis happens with DateTime.Now and DateTime.UtcNow\r\n\r\nI am unable to reproduce these steps on the current main branch.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 210217148, "label": "Rewrote network stack"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/90#issuecomment-282635696", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/90", "id": 282635696, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MjYzNTY5Ng==", "user": {"value": 12808515, "label": "Zegeger"}, "created_at": "2017-02-27T06:03:32Z", "updated_at": "2017-02-27T06:03:32Z", "author_association": "CONTRIBUTOR", "body": "Applied requested changes and I believe fixed the crash issues.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 210217148, "label": "Rewrote network stack"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/93#issuecomment-282724765", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/93", "id": 282724765, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MjcyNDc2NQ==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-27T13:51:26Z", "updated_at": "2017-02-27T13:51:26Z", "author_association": "MEMBER", "body": "see #94 ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 210350061, "label": "Support easy permissions in ACEmulator"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/84#issuecomment-282728268", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/84", "id": 282728268, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MjcyODI2OA==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-02-27T14:03:07Z", "updated_at": "2017-02-27T14:03:07Z", "author_association": "CONTRIBUTOR", "body": "Fantoms, we had that giant changeset for the Network overhaul.  Rebase this and give it another go and we'll merge it in.", "reactions": "{\"total_count\": 1, \"+1\": 1, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 209354215, "label": "Skill spend upper bound check too prevent xp crash, also Max Skills!"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/92#issuecomment-282737851", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/92", "id": 282737851, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MjczNzg1MQ==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-02-27T14:38:23Z", "updated_at": "2017-02-27T14:38:23Z", "author_association": "MEMBER", "body": "See #95 ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 210289130, "label": "Add in support for the old global Admin channels"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/83#issuecomment-282738707", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/83", "id": 282738707, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MjczODcwNw==", "user": {"value": 12808515, "label": "Zegeger"}, "created_at": "2017-02-27T14:41:26Z", "updated_at": "2017-02-27T14:41:26Z", "author_association": "CONTRIBUTOR", "body": "Completed in PR 90\r\nNetworkBuffer became NetworkSession", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 209201688, "label": "Add message buffering and send as multifragment packet"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/84#issuecomment-283558190", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/84", "id": 283558190, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MzU1ODE5MA==", "user": {"value": 37453, "label": "fantoms"}, "created_at": "2017-03-02T05:11:09Z", "updated_at": "2017-03-02T05:11:09Z", "author_association": "CONTRIBUTOR", "body": "I believe this is ready for a another eye if it is worth having Skills raise to max skill level.\r\n\r\nThe abilities are not working; example: when you go above 221 on strength.\r\n\r\nHere is a packet capture showing the abilities issue: http://pastebin.com/Bn5Sjew6\r\n\r\nThanks!", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 209354215, "label": "Skill spend upper bound check too prevent xp crash, also Max Skills!"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/84#issuecomment-283644641", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/84", "id": 283644641, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MzY0NDY0MQ==", "user": {"value": 25351661, "label": "Mogwai-TheFurry"}, "created_at": "2017-03-02T12:47:49Z", "updated_at": "2017-03-02T12:47:49Z", "author_association": "CONTRIBUTOR", "body": "102 files changed?  I can't tell what's your PR vs what was a merge.  Can you create a new branch and cherry pick the commits that have your code and resubmit?  I just have no way of telling what was yours versus what was merged stuff.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 209354215, "label": "Skill spend upper bound check too prevent xp crash, also Max Skills!"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/84#issuecomment-283647919", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/84", "id": 283647919, "node_id": "MDEyOklzc3VlQ29tbWVudDI4MzY0NzkxOQ==", "user": {"value": 37453, "label": "fantoms"}, "created_at": "2017-03-02T13:03:43Z", "updated_at": "2017-03-02T13:03:43Z", "author_association": "CONTRIBUTOR", "body": "I will start over on this PR, the changes are minimal but a rebase did not work as expected.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 209354215, "label": "Skill spend upper bound check too prevent xp crash, also Max Skills!"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/107#issuecomment-284311511", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/107", "id": 284311511, "node_id": "MDEyOklzc3VlQ29tbWVudDI4NDMxMTUxMQ==", "user": {"value": 10137, "label": "ghost"}, "created_at": "2017-03-06T06:13:51Z", "updated_at": "2017-03-06T06:13:51Z", "author_association": "NONE", "body": "So the file names themselves would be hard-coded and just pull in the directory location from the config file, thereby allowing the two config directives to condense into one.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 211999671, "label": "Add DAT loading into main process"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/107#issuecomment-284312826", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/107", "id": 284312826, "node_id": "MDEyOklzc3VlQ29tbWVudDI4NDMxMjgyNg==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-03-06T06:25:16Z", "updated_at": "2017-03-06T06:25:16Z", "author_association": "MEMBER", "body": "Exactly. I can't imagine a reason why the file names couldn't or shouldn't be hardcoded, but I can think of a few reasons why one might want to change the directory location. One of which could be having multiple versions of DAT patches to switch between perhaps?", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 211999671, "label": "Add DAT loading into main process"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/56#issuecomment-284313391", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/56", "id": 284313391, "node_id": "MDEyOklzc3VlQ29tbWVudDI4NDMxMzM5MQ==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-03-06T06:29:28Z", "updated_at": "2017-03-06T06:29:28Z", "author_association": "MEMBER", "body": "This issue possibly links with a TODO in #108 ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 206955987, "label": "Background maintenance thread"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/107#issuecomment-284326453", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/107", "id": 284326453, "node_id": "MDEyOklzc3VlQ29tbWVudDI4NDMyNjQ1Mw==", "user": {"value": 10137, "label": "ghost"}, "created_at": "2017-03-06T07:59:00Z", "updated_at": "2017-03-06T07:59:00Z", "author_association": "NONE", "body": "Stupid tab vs spaces kept tripping me up.  I also noticed that I missed a whole file that should have been included in the original commit.  'git add <filename>, ftw.  It has now been added into the PR with the changes to the config variable(s).", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 211999671, "label": "Add DAT loading into main process"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/109#issuecomment-284556139", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/109", "id": 284556139, "node_id": "MDEyOklzc3VlQ29tbWVudDI4NDU1NjEzOQ==", "user": {"value": 25387933, "label": "Forbiddenz"}, "created_at": "2017-03-06T22:33:12Z", "updated_at": "2017-03-06T22:33:12Z", "author_association": "CONTRIBUTOR", "body": "Fixed,", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 212215798, "label": "World Object Fix"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/21#issuecomment-285084104", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/21", "id": 285084104, "node_id": "MDEyOklzc3VlQ29tbWVudDI4NTA4NDEwNA==", "user": {"value": 25650194, "label": "Thwargle"}, "created_at": "2017-03-08T16:07:10Z", "updated_at": "2017-03-08T16:07:10Z", "author_association": "NONE", "body": "Technically; we DO have a launcher. I will work with whoever necessary to implement this into the launcher when ready. It's currently stable and fully featured as far as i can tell; so it's at a stage where implementing true password passing between server and client launch is reasonable.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 205361124, "label": "Change password hashing to use scrypt"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/58#issuecomment-285278550", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/58", "id": 285278550, "node_id": "MDEyOklzc3VlQ29tbWVudDI4NTI3ODU1MA==", "user": {"value": 11369233, "label": "LtRipley36706"}, "created_at": "2017-03-09T07:44:47Z", "updated_at": "2017-03-09T07:44:47Z", "author_association": "MEMBER", "body": "PR #116 addresses the last change this PR was working on.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 207015535, "label": "Added support on ACE console for setting per character admin rights and character renaming"}}
{"html_url": "https://github.com/ACEmulator/ACE/issues/52#issuecomment-285877915", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/52", "id": 285877915, "node_id": "MDEyOklzc3VlQ29tbWVudDI4NTg3NzkxNQ==", "user": {"value": 37453, "label": "fantoms"}, "created_at": "2017-03-11T16:13:01Z", "updated_at": "2017-03-11T16:53:37Z", "author_association": "CONTRIBUTOR", "body": "Please review Pull Request #119, which should now close most of the requirements for this issue.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 206403292, "label": "Leveling up when granted xp"}}
{"html_url": "https://github.com/ACEmulator/ACE/pull/118#issuecomment-285880069", "issue_url": "https://api.github.com/repos/ACEmulator/ACE/issues/118", "id": 285880069, "node_id": "MDEyOklzc3VlQ29tbWVudDI4NTg4MDA2OQ==", "user": {"value": 26334665, "label": "lurker-mcdoogle"}, "created_at": "2017-03-11T16:47:30Z", "updated_at": "2017-03-11T16:47:30Z", "author_association": "CONTRIBUTOR", "body": "Hrm, noticed some bugs around objects not getting into the landblock collection. Not mergeworthy yet, but feel free to comment. ", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "performed_via_github_app": null, "issue": {"value": 213533094, "label": "Landblocks"}}