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/issues/53#issuecomment-285880346,https://api.github.com/repos/ACEmulator/ACE/issues/53,285880346,MDEyOklzc3VlQ29tbWVudDI4NTg4MDM0Ng==,37453,2017-03-11T16:52:00Z,2017-03-11T16:52:00Z,CONTRIBUTOR,"Added skill credits and ability to grant Skill credits on level up, in Pull Request #119.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,206403348 https://github.com/ACEmulator/ACE/pull/118#issuecomment-285907205,https://api.github.com/repos/ACEmulator/ACE/issues/118,285907205,MDEyOklzc3VlQ29tbWVudDI4NTkwNzIwNQ==,10608427,2017-03-11T22:59:59Z,2017-03-11T22:59:59Z,MEMBER,Is this a modified version of mogwais code?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213533094 https://github.com/ACEmulator/ACE/pull/118#issuecomment-285913728,https://api.github.com/repos/ACEmulator/ACE/issues/118,285913728,MDEyOklzc3VlQ29tbWVudDI4NTkxMzcyOA==,26334665,2017-03-12T01:18:42Z,2017-03-12T01:18:42Z,CONTRIBUTOR,"no, he was talking about event systems of some sort.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213533094 https://github.com/ACEmulator/ACE/pull/119#issuecomment-285913800,https://api.github.com/repos/ACEmulator/ACE/issues/119,285913800,MDEyOklzc3VlQ29tbWVudDI4NTkxMzgwMA==,37453,2017-03-12T01:20:16Z,2017-03-12T05:23:25Z,CONTRIBUTOR,"-Added the max level effect -Moved the effects to after the level-up network message sequence -I believe made the /grantxp now command work properly. Please let me know if this is correct: There was an issue where the max xp was not locking to Infinity, so I had to add another GrantXp command to add Additional XP for skill leveling, for after max level is achieved. -Moved the available credit update to the end so it doesn't spam the message until all credits are achieved, this fixed a bug I added.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213538178 https://github.com/ACEmulator/ACE/pull/118#issuecomment-285917933,https://api.github.com/repos/ACEmulator/ACE/issues/118,285917933,MDEyOklzc3VlQ29tbWVudDI4NTkxNzkzMw==,26334665,2017-03-12T02:54:11Z,2017-03-12T02:54:11Z,CONTRIBUTOR,"Updated with: 1) basic object factories 2) log4net - not having console output was killing me and we needed a logging framework anyway. nuget to the rescue. 3) moved ""in front of"" calculations to hang off the Position object for better reuse 4) fixed various landblock object cache bugs 5) added even more // TODO","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213533094 https://github.com/ACEmulator/ACE/pull/119#issuecomment-285923241,https://api.github.com/repos/ACEmulator/ACE/issues/119,285923241,MDEyOklzc3VlQ29tbWVudDI4NTkyMzI0MQ==,37453,2017-03-12T05:20:00Z,2017-03-12T05:20:00Z,CONTRIBUTOR,"Known issue: No formal name or label on skills. Examples: ""VoidMagic"" or ""SneakAttack"".","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213538178 https://github.com/ACEmulator/ACE/pull/117#issuecomment-285928441,https://api.github.com/repos/ACEmulator/ACE/issues/117,285928441,MDEyOklzc3VlQ29tbWVudDI4NTkyODQ0MQ==,11369233,2017-03-12T07:49:11Z,2017-03-12T07:49:11Z,MEMBER,Working on a few changes,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213281968 https://github.com/ACEmulator/ACE/pull/118#issuecomment-285936751,https://api.github.com/repos/ACEmulator/ACE/issues/118,285936751,MDEyOklzc3VlQ29tbWVudDI4NTkzNjc1MQ==,10608427,2017-03-12T10:53:22Z,2017-03-12T10:53:22Z,MEMBER,"Ahh ok. I'll have to review this today. Talking with mogwai in private, his initial plan was event based, but it didn't pan out, so he scrapped that and was working on a different system, so I wasn't sure if this was that.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213533094 https://github.com/ACEmulator/ACE/pull/118#issuecomment-285966645,https://api.github.com/repos/ACEmulator/ACE/issues/118,285966645,MDEyOklzc3VlQ29tbWVudDI4NTk2NjY0NQ==,10608427,2017-03-12T19:04:57Z,2017-03-12T19:04:57Z,MEMBER,"I loaded this up. Broadcasting works which is nice. Seems to be a bit of client and server instability issues though. Will have to look at it more in depth. Also, it would be cleaner to review if the log4net changes weren't combined with the landblock code. Landblock management is already complex enough.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213533094 https://github.com/ACEmulator/ACE/pull/118#issuecomment-285994516,https://api.github.com/repos/ACEmulator/ACE/issues/118,285994516,MDEyOklzc3VlQ29tbWVudDI4NTk5NDUxNg==,26334665,2017-03-13T01:18:29Z,2017-03-13T01:18:29Z,CONTRIBUTOR,"I can gut the log4net stuff if you want, but I couldn't have debugged this without it. Either way. And yeah, I had stability issues as well, but it wasn't anything consistently reproducible. There was a comment about the update having to be sent after the client entered portal space, but I didn't do anything explicit to account for it. Let me know if you find anything specific.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213533094 https://github.com/ACEmulator/ACE/pull/118#issuecomment-285997814,https://api.github.com/repos/ACEmulator/ACE/issues/118,285997814,MDEyOklzc3VlQ29tbWVudDI4NTk5NzgxNA==,10608427,2017-03-13T01:53:40Z,2017-03-13T01:53:40Z,MEMBER,"At this point, might as well keep the log4net in there. It makes sense moving forward, and no sense in duplicating the work later.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213533094 https://github.com/ACEmulator/ACE/pull/118#issuecomment-285998186,https://api.github.com/repos/ACEmulator/ACE/issues/118,285998186,MDEyOklzc3VlQ29tbWVudDI4NTk5ODE4Ng==,26334665,2017-03-13T01:57:25Z,2017-03-13T01:57:25Z,CONTRIBUTOR,Merge conflicts now. Closing - will rebase when i can.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213533094 https://github.com/ACEmulator/ACE/pull/117#issuecomment-286009596,https://api.github.com/repos/ACEmulator/ACE/issues/117,286009596,MDEyOklzc3VlQ29tbWVudDI4NjAwOTU5Ng==,11369233,2017-03-13T03:48:59Z,2017-03-13T03:48:59Z,MEMBER,moving stuff around and redoing PR,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213281968 https://github.com/ACEmulator/ACE/issues/19#issuecomment-286120297,https://api.github.com/repos/ACEmulator/ACE/issues/19,286120297,MDEyOklzc3VlQ29tbWVudDI4NjEyMDI5Nw==,10608427,2017-03-13T14:18:26Z,2017-03-13T14:18:26Z,MEMBER,"Still need to add the movement event that gets sent to the client putting the client in the logoff animation. Hopefully this stops the sounds from continuing through to character selection screen. Also need to broadcast the event to others.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,205346915 https://github.com/ACEmulator/ACE/issues/51#issuecomment-286124129,https://api.github.com/repos/ACEmulator/ACE/issues/51,286124129,MDEyOklzc3VlQ29tbWVudDI4NjEyNDEyOQ==,37453,2017-03-13T14:30:57Z,2017-03-13T14:30:57Z,CONTRIBUTOR,"A small portion of saving the character is available in Pull Request #125, as it will save the character properties to the database.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,206403228 https://github.com/ACEmulator/ACE/pull/126#issuecomment-286517825,https://api.github.com/repos/ACEmulator/ACE/issues/126,286517825,MDEyOklzc3VlQ29tbWVudDI4NjUxNzgyNQ==,26334665,2017-03-14T18:32:43Z,2017-03-14T18:32:43Z,CONTRIBUTOR,Reopening with squashed commits.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213875928 https://github.com/ACEmulator/ACE/pull/131#issuecomment-286959999,https://api.github.com/repos/ACEmulator/ACE/issues/131,286959999,MDEyOklzc3VlQ29tbWVudDI4Njk1OTk5OQ==,25460553,2017-03-16T05:13:40Z,2017-03-16T05:13:40Z,CONTRIBUTOR,I think I did something wrong. Going to close this for now.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,214596925 https://github.com/ACEmulator/ACE/pull/136#issuecomment-287145676,https://api.github.com/repos/ACEmulator/ACE/issues/136,287145676,MDEyOklzc3VlQ29tbWVudDI4NzE0NTY3Ng==,25460553,2017-03-16T18:13:36Z,2017-03-16T18:13:36Z,CONTRIBUTOR,Ok - thanks for checking it out. I had to get a toe wet somehow ... ;),"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,214781043 https://github.com/ACEmulator/ACE/pull/137#issuecomment-287150964,https://api.github.com/repos/ACEmulator/ACE/issues/137,287150964,MDEyOklzc3VlQ29tbWVudDI4NzE1MDk2NA==,25387933,2017-03-16T18:32:12Z,2017-03-16T18:32:12Z,CONTRIBUTOR,Cleaned up ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,214784803 https://github.com/ACEmulator/ACE/pull/137#issuecomment-287152244,https://api.github.com/repos/ACEmulator/ACE/issues/137,287152244,MDEyOklzc3VlQ29tbWVudDI4NzE1MjI0NA==,25351661,2017-03-16T18:36:49Z,2017-03-16T18:36:49Z,CONTRIBUTOR,MutableWorldObject xml doc header....,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,214784803 https://github.com/ACEmulator/ACE/pull/125#issuecomment-287183202,https://api.github.com/repos/ACEmulator/ACE/issues/125,287183202,MDEyOklzc3VlQ29tbWVudDI4NzE4MzIwMg==,37453,2017-03-16T20:35:43Z,2017-03-16T20:35:43Z,CONTRIBUTOR,Network Conflicts resolved.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213680800 https://github.com/ACEmulator/ACE/pull/136#issuecomment-287223448,https://api.github.com/repos/ACEmulator/ACE/issues/136,287223448,MDEyOklzc3VlQ29tbWVudDI4NzIyMzQ0OA==,25460553,2017-03-16T23:29:38Z,2017-03-16T23:29:38Z,CONTRIBUTOR,Closing and resubmitting,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,214781043 https://github.com/ACEmulator/ACE/pull/136#issuecomment-287223477,https://api.github.com/repos/ACEmulator/ACE/issues/136,287223477,MDEyOklzc3VlQ29tbWVudDI4NzIyMzQ3Nw==,25460553,2017-03-16T23:29:47Z,2017-03-16T23:29:47Z,CONTRIBUTOR,"Closing and resubmitting ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,214781043 https://github.com/ACEmulator/ACE/pull/121#issuecomment-287252812,https://api.github.com/repos/ACEmulator/ACE/issues/121,287252812,MDEyOklzc3VlQ29tbWVudDI4NzI1MjgxMg==,26334665,2017-03-17T02:51:34Z,2017-03-17T02:51:34Z,CONTRIBUTOR,Known Issue: Adjacency matrix has issues broadcasting to the correct adjacencies. Shouldn't be a big deal - I'll fix it later.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213657914 https://github.com/ACEmulator/ACE/pull/121#issuecomment-287511136,https://api.github.com/repos/ACEmulator/ACE/issues/121,287511136,MDEyOklzc3VlQ29tbWVudDI4NzUxMTEzNg==,10608427,2017-03-18T02:44:38Z,2017-03-18T02:44:38Z,MEMBER,"Landblock.cs, line 289. Needs to be changed from > to >= movedObjects = movedObjects.Where(p => p.LastUpdatedTicks > p.LastMovementBroadcastTicks).ToList(); // flag them as updated now in order to reduce chance of missing an update movedObjects.ForEach(m => m.LastMovementBroadcastTicks = WorldManager.PortalYearTicks); What's happening is a player is moving on the same PortalYearTick, right before Landblock.UseTime is called. When this happens, the > fails, and the cobject isn't processed as a move. That move gets skipped over. Even though you set the LastMove to PortalYearTicks to indicate the object as processed, the next time UseTime is called, it's impossible for PortalYearTicks to have the same value as the previous call, thus, the >= won't be called in duplicate/error.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213657914 https://github.com/ACEmulator/ACE/pull/119#issuecomment-287548109,https://api.github.com/repos/ACEmulator/ACE/issues/119,287548109,MDEyOklzc3VlQ29tbWVudDI4NzU0ODEwOQ==,37453,2017-03-18T13:59:42Z,2017-03-18T13:59:42Z,CONTRIBUTOR,"Closing now, this was merged into PR #141 ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213538178 https://github.com/ACEmulator/ACE/pull/125#issuecomment-287548140,https://api.github.com/repos/ACEmulator/ACE/issues/125,287548140,MDEyOklzc3VlQ29tbWVudDI4NzU0ODE0MA==,37453,2017-03-18T14:00:13Z,2017-03-18T14:00:13Z,CONTRIBUTOR,"Closing now, this was merged into PR #141 ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213680800 https://github.com/ACEmulator/ACE/pull/134#issuecomment-287548162,https://api.github.com/repos/ACEmulator/ACE/issues/134,287548162,MDEyOklzc3VlQ29tbWVudDI4NzU0ODE2Mg==,37453,2017-03-18T14:00:37Z,2017-03-18T14:00:37Z,CONTRIBUTOR,"Closing now, since this was merged into PR #141 ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,214728678 https://github.com/ACEmulator/ACE/pull/121#issuecomment-287548276,https://api.github.com/repos/ACEmulator/ACE/issues/121,287548276,MDEyOklzc3VlQ29tbWVudDI4NzU0ODI3Ng==,10608427,2017-03-18T14:02:39Z,2017-03-18T14:02:39Z,MEMBER,"Landblock.cs UseTime() needs a Thread.Sleep(1) as the last line of the While() loop. We also have variables that are being referenced across multiple threads. WorldObject.LastUpdateTicks and MutableWorldObject.LastMovementBroadcastTicks... I'm gonna think about this some more today.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,213657914 https://github.com/ACEmulator/ACE/pull/141#issuecomment-287569603,https://api.github.com/repos/ACEmulator/ACE/issues/141,287569603,MDEyOklzc3VlQ29tbWVudDI4NzU2OTYwMw==,11369233,2017-03-18T19:40:04Z,2017-03-18T19:40:04Z,MEMBER,I broke this PR in an attempt to rebase off of current master.. My git-fu needs improvement.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,215172361 https://github.com/ACEmulator/ACE/issues/53#issuecomment-287586760,https://api.github.com/repos/ACEmulator/ACE/issues/53,287586760,MDEyOklzc3VlQ29tbWVudDI4NzU4Njc2MA==,37453,2017-03-19T01:10:40Z,2017-03-19T01:10:40Z,CONTRIBUTOR,"If PR #143 gets accepted, then I have a follow-up PR that will allow skill credits to be spent. I've already tested the code and it should allow for skill credit spending, saving, and loading.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,206403348 https://github.com/ACEmulator/ACE/pull/146#issuecomment-287591909,https://api.github.com/repos/ACEmulator/ACE/issues/146,287591909,MDEyOklzc3VlQ29tbWVudDI4NzU5MTkwOQ==,25460553,2017-03-19T03:37:25Z,2017-03-19T03:37:25Z,CONTRIBUTOR,"Jesus - I tried to squash it down to 1 commit and it went from 7 to 9. I will get git - but damn it is kicking my ass right now. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,215238775 https://github.com/ACEmulator/ACE/pull/148#issuecomment-287667143,https://api.github.com/repos/ACEmulator/ACE/issues/148,287667143,MDEyOklzc3VlQ29tbWVudDI4NzY2NzE0Mw==,25351661,2017-03-20T02:13:35Z,2017-03-20T02:13:35Z,CONTRIBUTOR,"So, here's the thing... Devs that know what to do with Docker don't need this - we can run scripts just fine. Players that want to host their own servers don't even have a clue what Docker is. Plus, you're linking to a docker image offsite that we can't control. We don't think this is necessary, but in the end we might host a db image on acemulator.org when we cut releases. I just don't think we're at a point where this is needed. Thanks!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,215282644 https://github.com/ACEmulator/ACE/pull/21#issuecomment-287667649,https://api.github.com/repos/ACEmulator/ACE/issues/21,287667649,MDEyOklzc3VlQ29tbWVudDI4NzY2NzY0OQ==,25351661,2017-03-20T02:20:14Z,2017-03-20T02:20:14Z,CONTRIBUTOR,closing due to being outdated plus inability to merge.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,205361124 https://github.com/ACEmulator/ACE/issues/51#issuecomment-287670049,https://api.github.com/repos/ACEmulator/ACE/issues/51,287670049,MDEyOklzc3VlQ29tbWVudDI4NzY3MDA0OQ==,11369233,2017-03-20T02:53:25Z,2017-03-20T02:53:25Z,MEMBER,Saving intermittently and on logout is in.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,206403228 https://github.com/ACEmulator/ACE/pull/148#issuecomment-287670070,https://api.github.com/repos/ACEmulator/ACE/issues/148,287670070,MDEyOklzc3VlQ29tbWVudDI4NzY3MDA3MA==,8466632,2017-03-20T02:53:36Z,2017-03-20T02:53:36Z,NONE,"No problem, Since i'm using it, i thought maybe you guys might find it useful. I'll find other way to contribute :)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,215282644 https://github.com/ACEmulator/ACE/issues/52#issuecomment-287670094,https://api.github.com/repos/ACEmulator/ACE/issues/52,287670094,MDEyOklzc3VlQ29tbWVudDI4NzY3MDA5NA==,11369233,2017-03-20T02:54:03Z,2017-03-20T02:54:03Z,MEMBER,@fantoms completed this and it was merged in.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,206403292 https://github.com/ACEmulator/ACE/pull/156#issuecomment-287852886,https://api.github.com/repos/ACEmulator/ACE/issues/156,287852886,MDEyOklzc3VlQ29tbWVudDI4Nzg1Mjg4Ng==,37453,2017-03-20T18:23:52Z,2017-03-20T18:23:52Z,CONTRIBUTOR,"Tested this utilizing multiple characters and passes were successful. Aside from this code, your command really helps points out the bug/non-issue in the client, where your total health is greater than the actual health - Example - In-game action sequence after Spending one rank up, for endurance (Total 61): 1. Health Total: `1/31` 2. type: `@heal` 3. Health Total: `30/31`","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,215484872 https://github.com/ACEmulator/ACE/pull/148#issuecomment-287964793,https://api.github.com/repos/ACEmulator/ACE/issues/148,287964793,MDEyOklzc3VlQ29tbWVudDI4Nzk2NDc5Mw==,1765994,2017-03-21T03:14:49Z,2017-03-21T03:14:49Z,CONTRIBUTOR,"If we had the actual source and wanted to move it away from SQL Server and Windows Server, then deploying each piece (Database, GLS, etc) in a docker would have provided a great way to link all of the components. Then have a VM OVF/OVA with CentOS or similar OS and have it load the dockers would have been a decent deployment model. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,215282644 https://github.com/ACEmulator/ACE/pull/163#issuecomment-288315607,https://api.github.com/repos/ACEmulator/ACE/issues/163,288315607,MDEyOklzc3VlQ29tbWVudDI4ODMxNTYwNw==,11369233,2017-03-22T07:05:10Z,2017-03-22T07:05:10Z,MEMBER,"Now, really, all good here 👍 ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,215964624 https://github.com/ACEmulator/ACE/pull/167#issuecomment-288825995,https://api.github.com/repos/ACEmulator/ACE/issues/167,288825995,MDEyOklzc3VlQ29tbWVudDI4ODgyNTk5NQ==,11369233,2017-03-23T18:56:54Z,2017-03-23T18:56:54Z,MEMBER,Client currently crashes when the db objects are sent to it...,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,216282064 https://github.com/ACEmulator/ACE/pull/167#issuecomment-289094900,https://api.github.com/repos/ACEmulator/ACE/issues/167,289094900,MDEyOklzc3VlQ29tbWVudDI4OTA5NDkwMA==,11369233,2017-03-24T17:46:38Z,2017-03-24T17:46:38Z,MEMBER,"Lifestones.. Lifestones everywhere! ![139vy](https://cloud.githubusercontent.com/assets/11369233/24306810/442826f4-1098-11e7-827d-edb4621c05e5.jpg) ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,216282064 https://github.com/ACEmulator/ACE/pull/167#issuecomment-289177445,https://api.github.com/repos/ACEmulator/ACE/issues/167,289177445,MDEyOklzc3VlQ29tbWVudDI4OTE3NzQ0NQ==,25518589,2017-03-25T01:13:54Z,2017-03-25T01:13:54Z,CONTRIBUTOR,"Works for me, too Mogwai. Exciting to see this materialize through the process. Generalized object factory is going to quickly open a lot of other doors as well. Thanks for the effort! ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,216282064 https://github.com/ACEmulator/ACE/pull/160#issuecomment-289435283,https://api.github.com/repos/ACEmulator/ACE/issues/160,289435283,MDEyOklzc3VlQ29tbWVudDI4OTQzNTI4Mw==,25351661,2017-03-27T12:12:32Z,2017-03-27T12:12:32Z,CONTRIBUTOR,"Sat down to review all this, but there's 5 merge conflicts. If you could take a look at rebasing with the big item / database overhaul that would be great.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,215897071 https://github.com/ACEmulator/ACE/pull/160#issuecomment-289553473,https://api.github.com/repos/ACEmulator/ACE/issues/160,289553473,MDEyOklzc3VlQ29tbWVudDI4OTU1MzQ3Mw==,26508216,2017-03-27T19:06:37Z,2017-03-27T19:06:37Z,CONTRIBUTOR,Thanks for the review. I'll redo this with the new database.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,215897071 https://github.com/ACEmulator/ACE/issues/166#issuecomment-289575765,https://api.github.com/repos/ACEmulator/ACE/issues/166,289575765,MDEyOklzc3VlQ29tbWVudDI4OTU3NTc2NQ==,25460553,2017-03-27T20:28:27Z,2017-03-27T20:28:27Z,CONTRIBUTOR,This is inprogress,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,216250810 https://github.com/ACEmulator/ACE/pull/172#issuecomment-289590239,https://api.github.com/repos/ACEmulator/ACE/issues/172,289590239,MDEyOklzc3VlQ29tbWVudDI4OTU5MDIzOQ==,26508216,2017-03-27T21:22:33Z,2017-03-27T21:22:33Z,CONTRIBUTOR,The last commit (b5480f2) has the requested changes,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,217055250 https://github.com/ACEmulator/ACE/issues/180#issuecomment-289746755,https://api.github.com/repos/ACEmulator/ACE/issues/180,289746755,MDEyOklzc3VlQ29tbWVudDI4OTc0Njc1NQ==,25351661,2017-03-28T11:57:18Z,2017-03-28T11:57:18Z,CONTRIBUTOR,"Problem is we don't have anything in place to differentiate between walking and TPing within a landblock. Shouldn't be hard, though.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,217470832 https://github.com/ACEmulator/ACE/pull/182#issuecomment-290084445,https://api.github.com/repos/ACEmulator/ACE/issues/182,290084445,MDEyOklzc3VlQ29tbWVudDI5MDA4NDQ0NQ==,25460553,2017-03-29T13:09:24Z,2017-03-29T13:09:24Z,CONTRIBUTOR,"In looking at this this morning - I think I see what the issue is. I was working on this right before someone was doing some work on animation. I had started down a path that I needed to create a properly formed F74C MovementEvent. He and I both created objects and enums that were the same but named differently. Both are in here and I am now sending a bogus F74 message. That is why my neckid dude will not bend down to drop the wand. I won't be able to untangle this until later today - but anyone that has time, other than that message being fubar - the implementation of the container inventory dictionary, the landblock logic for dropping an item - it would be helpful to know if I am on the right track with how it is implemented. Thanks ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,217729589 https://github.com/ACEmulator/ACE/pull/182#issuecomment-290238147,https://api.github.com/repos/ACEmulator/ACE/issues/182,290238147,MDEyOklzc3VlQ29tbWVudDI5MDIzODE0Nw==,25460553,2017-03-29T21:53:56Z,2017-03-29T21:53:56Z,CONTRIBUTOR,This is fubar. Going to fix it and resubmit.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,217729589 https://github.com/ACEmulator/ACE/pull/183#issuecomment-290313862,https://api.github.com/repos/ACEmulator/ACE/issues/183,290313862,MDEyOklzc3VlQ29tbWVudDI5MDMxMzg2Mg==,11369233,2017-03-30T06:22:39Z,2017-03-30T06:22:39Z,MEMBER,"Getting several crashes with the current db load in, needs a bit more pruning of objects as there are lots of player created objects in this db in addition to static objects, which probably leads to crashing in some of housing settlements or towns","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,217957579 https://github.com/ACEmulator/ACE/pull/184#issuecomment-290456123,https://api.github.com/repos/ACEmulator/ACE/issues/184,290456123,MDEyOklzc3VlQ29tbWVudDI5MDQ1NjEyMw==,37453,2017-03-30T15:56:12Z,2017-03-30T15:56:12Z,CONTRIBUTOR,"I am going to close this PR, untill I work the Positions and CharacterPositions in too the same object and the code ready for another look. Thanks For your reviews so far!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218060844 https://github.com/ACEmulator/ACE/pull/186#issuecomment-290590681,https://api.github.com/repos/ACEmulator/ACE/issues/186,290590681,MDEyOklzc3VlQ29tbWVudDI5MDU5MDY4MQ==,25460553,2017-03-31T01:28:25Z,2017-03-31T01:28:25Z,CONTRIBUTOR,"I made the requested changes. Let me know if you see anything else. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218255708 https://github.com/ACEmulator/ACE/pull/186#issuecomment-290596235,https://api.github.com/repos/ACEmulator/ACE/issues/186,290596235,MDEyOklzc3VlQ29tbWVudDI5MDU5NjIzNQ==,25460553,2017-03-31T02:11:37Z,2017-03-31T02:11:37Z,CONTRIBUTOR,Done.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218255708 https://github.com/ACEmulator/ACE/pull/187#issuecomment-290756995,https://api.github.com/repos/ACEmulator/ACE/issues/187,290756995,MDEyOklzc3VlQ29tbWVudDI5MDc1Njk5NQ==,25387933,2017-03-31T16:11:52Z,2017-03-31T16:11:52Z,CONTRIBUTOR,Updated,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218334549 https://github.com/ACEmulator/ACE/pull/187#issuecomment-290764066,https://api.github.com/repos/ACEmulator/ACE/issues/187,290764066,MDEyOklzc3VlQ29tbWVudDI5MDc2NDA2Ng==,25387933,2017-03-31T16:38:58Z,2017-03-31T16:38:58Z,CONTRIBUTOR,"Removed some useless comments I noticed too. - Spell is now a mutable object","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218334549 https://github.com/ACEmulator/ACE/pull/187#issuecomment-290821532,https://api.github.com/repos/ACEmulator/ACE/issues/187,290821532,MDEyOklzc3VlQ29tbWVudDI5MDgyMTUzMg==,25387933,2017-03-31T20:28:58Z,2017-03-31T20:28:58Z,CONTRIBUTOR,Spells ! We have some fun spells.. I will work to improve it but this is worth it to submit it now as it should works in multiplayer for giggles..,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218334549 https://github.com/ACEmulator/ACE/pull/187#issuecomment-290918730,https://api.github.com/repos/ACEmulator/ACE/issues/187,290918730,MDEyOklzc3VlQ29tbWVudDI5MDkxODczMA==,25387933,2017-04-01T13:00:41Z,2017-04-01T13:00:41Z,CONTRIBUTOR,"Closing this, I found some issues with it. I also am thinking we need some kind of naming convention between Entity and Enum.. Starting to Get confusing.. IE PlayerEntity, SpellEntity, Etc as we build more entitys.. enums should just be Spell, Player, etc as needed.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218334549 https://github.com/ACEmulator/ACE/pull/191#issuecomment-290962833,https://api.github.com/repos/ACEmulator/ACE/issues/191,290962833,MDEyOklzc3VlQ29tbWVudDI5MDk2MjgzMw==,25460553,2017-04-02T03:47:07Z,2017-04-02T03:47:07Z,CONTRIBUTOR,No conflicts now and this does fix a serious bug. Let me know what you think.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218735283 https://github.com/ACEmulator/ACE/pull/189#issuecomment-290962849,https://api.github.com/repos/ACEmulator/ACE/issues/189,290962849,MDEyOklzc3VlQ29tbWVudDI5MDk2Mjg0OQ==,25460553,2017-04-02T03:47:46Z,2017-04-02T03:47:46Z,CONTRIBUTOR,I will fix this first thing in the morning. I looked they are easy to resolve. Thanks for checking it over.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218616162 https://github.com/ACEmulator/ACE/pull/194#issuecomment-290963680,https://api.github.com/repos/ACEmulator/ACE/issues/194,290963680,MDEyOklzc3VlQ29tbWVudDI5MDk2MzY4MA==,37453,2017-04-02T04:18:08Z,2017-04-02T04:18:08Z,CONTRIBUTOR,"In my defense, here is the previously requested change: https://github.com/ACEmulator/ACE/pull/184/files/0fda5e079b51c574b0565d7e01fa51e8548d5d0b","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218740683 https://github.com/ACEmulator/ACE/pull/190#issuecomment-290970418,https://api.github.com/repos/ACEmulator/ACE/issues/190,290970418,MDEyOklzc3VlQ29tbWVudDI5MDk3MDQxOA==,26508216,2017-04-02T07:41:29Z,2017-04-02T07:41:29Z,CONTRIBUTOR,Fixed the merge conflict.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218690610 https://github.com/ACEmulator/ACE/pull/189#issuecomment-290985538,https://api.github.com/repos/ACEmulator/ACE/issues/189,290985538,MDEyOklzc3VlQ29tbWVudDI5MDk4NTUzOA==,25460553,2017-04-02T13:12:04Z,2017-04-02T13:12:04Z,CONTRIBUTOR,"I fixed the merge issues. When you get a chance to review this - if you look at: public void HandleDropItem(ObjectGuid objectGuid, Session session) - I am not sure if it needs to be on world object or player. I cannot remember if any NPC's would drop anything as part of a quest. I put in in WO just in case. That might be a mistake. If it needs to go into player - I need to create a method on world object to update containerId. Anyway - any feedback is appreciated. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218616162 https://github.com/ACEmulator/ACE/pull/191#issuecomment-291004035,https://api.github.com/repos/ACEmulator/ACE/issues/191,291004035,MDEyOklzc3VlQ29tbWVudDI5MTAwNDAzNQ==,25460553,2017-04-02T18:15:44Z,2017-04-02T18:15:44Z,CONTRIBUTOR,Fixed last conflict,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218735283 https://github.com/ACEmulator/ACE/issues/144#issuecomment-291013240,https://api.github.com/repos/ACEmulator/ACE/issues/144,291013240,MDEyOklzc3VlQ29tbWVudDI5MTAxMzI0MA==,25460553,2017-04-02T20:37:35Z,2017-04-02T20:37:35Z,CONTRIBUTOR,If the first part gets checked in - I will implement pickup item - then add saving your inventory to the database for persistence. I will add inscription next.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,215224510 https://github.com/ACEmulator/ACE/pull/190#issuecomment-291120839,https://api.github.com/repos/ACEmulator/ACE/issues/190,291120839,MDEyOklzc3VlQ29tbWVudDI5MTEyMDgzOQ==,25351661,2017-04-03T11:49:54Z,2017-04-03T11:49:54Z,CONTRIBUTOR,I accepted Og's inventory PR which created conflicts again. Good news is you're now at the top of the queue this time!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218690610 https://github.com/ACEmulator/ACE/pull/183#issuecomment-291122387,https://api.github.com/repos/ACEmulator/ACE/issues/183,291122387,MDEyOklzc3VlQ29tbWVudDI5MTEyMjM4Nw==,25351661,2017-04-03T11:58:02Z,2017-04-03T11:58:02Z,CONTRIBUTOR,"Ok, I'll do the rebase without all the changes and go from there.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,217957579 https://github.com/ACEmulator/ACE/issues/4#issuecomment-291123944,https://api.github.com/repos/ACEmulator/ACE/issues/4,291123944,MDEyOklzc3VlQ29tbWVudDI5MTEyMzk0NA==,25351661,2017-04-03T12:06:09Z,2017-04-03T12:06:09Z,CONTRIBUTOR,mostly done last week. closing this for lack of more descriptive requirements.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,204489832 https://github.com/ACEmulator/ACE/pull/198#issuecomment-291271293,https://api.github.com/repos/ACEmulator/ACE/issues/198,291271293,MDEyOklzc3VlQ29tbWVudDI5MTI3MTI5Mw==,25351661,2017-04-03T20:55:57Z,2017-04-03T20:55:57Z,CONTRIBUTOR,"Any chance you can concat the portal files down to a single file for purposes of the PR? In the script directory, you should be able to do this (on Windows command line): copy *.sql AllPortals.sql We'd just need to know to run that via the SqlYog import tool rather than the query editor.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,219018504 https://github.com/ACEmulator/ACE/pull/190#issuecomment-291571075,https://api.github.com/repos/ACEmulator/ACE/issues/190,291571075,MDEyOklzc3VlQ29tbWVudDI5MTU3MTA3NQ==,26508216,2017-04-04T17:22:04Z,2017-04-04T17:22:04Z,CONTRIBUTOR,Fixed the merge conflicts,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,218690610 https://github.com/ACEmulator/ACE/issues/209#issuecomment-292063770,https://api.github.com/repos/ACEmulator/ACE/issues/209,292063770,MDEyOklzc3VlQ29tbWVudDI5MjA2Mzc3MA==,37453,2017-04-06T04:08:59Z,2017-04-06T04:08:59Z,CONTRIBUTOR,"After testing a few times, I believe that your commit b52abcd is the culprit. When I revert this change, rotations are restored.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,219614446 https://github.com/ACEmulator/ACE/issues/209#issuecomment-292071321,https://api.github.com/repos/ACEmulator/ACE/issues/209,292071321,MDEyOklzc3VlQ29tbWVudDI5MjA3MTMyMQ==,37453,2017-04-06T05:17:17Z,2017-04-06T05:17:17Z,CONTRIBUTOR,PR #214 seems to also fix the issue for me.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,219614446 https://github.com/ACEmulator/ACE/pull/211#issuecomment-292333744,https://api.github.com/repos/ACEmulator/ACE/issues/211,292333744,MDEyOklzc3VlQ29tbWVudDI5MjMzMzc0NA==,25460553,2017-04-06T21:47:06Z,2017-04-06T21:47:06Z,CONTRIBUTOR,I regressed with my latest changes. Going to close this for now.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,219773627 https://github.com/ACEmulator/ACE/pull/218#issuecomment-292709120,https://api.github.com/repos/ACEmulator/ACE/issues/218,292709120,MDEyOklzc3VlQ29tbWVudDI5MjcwOTEyMA==,26508216,2017-04-08T10:26:18Z,2017-04-08T10:26:18Z,CONTRIBUTOR,I'll close this one and will open a clean PR where I merged the master branch beforehand.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220357495 https://github.com/ACEmulator/ACE/issues/209#issuecomment-292803359,https://api.github.com/repos/ACEmulator/ACE/issues/209,292803359,MDEyOklzc3VlQ29tbWVudDI5MjgwMzM1OQ==,11369233,2017-04-09T18:31:16Z,2017-04-09T18:31:16Z,MEMBER,"agreed, this issue seems resolved. Closing for now.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,219614446 https://github.com/ACEmulator/ACE/pull/169#issuecomment-292815154,https://api.github.com/repos/ACEmulator/ACE/issues/169,292815154,MDEyOklzc3VlQ29tbWVudDI5MjgxNTE1NA==,11369233,2017-04-09T21:39:23Z,2017-04-09T21:39:23Z,MEMBER,See #223 ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,216958720 https://github.com/ACEmulator/ACE/pull/224#issuecomment-292830199,https://api.github.com/repos/ACEmulator/ACE/issues/224,292830199,MDEyOklzc3VlQ29tbWVudDI5MjgzMDE5OQ==,25460553,2017-04-10T01:40:28Z,2017-04-10T01:40:28Z,CONTRIBUTOR,Forgot to give a shoutout to Lidefeath for helping me with the lambda expression. ;),"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220517521 https://github.com/ACEmulator/ACE/pull/226#issuecomment-293031394,https://api.github.com/repos/ACEmulator/ACE/issues/226,293031394,MDEyOklzc3VlQ29tbWVudDI5MzAzMTM5NA==,37453,2017-04-10T18:07:14Z,2017-04-10T18:07:14Z,CONTRIBUTOR,"After pulling this patch, I was able to perform animations on different integers.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220709475 https://github.com/ACEmulator/ACE/issues/228#issuecomment-293108786,https://api.github.com/repos/ACEmulator/ACE/issues/228,293108786,MDEyOklzc3VlQ29tbWVudDI5MzEwODc4Ng==,11369233,2017-04-10T23:38:10Z,2017-04-10T23:38:10Z,MEMBER,Players that disconnect cleanly also persist in world,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220798868 https://github.com/ACEmulator/ACE/issues/231#issuecomment-293122342,https://api.github.com/repos/ACEmulator/ACE/issues/231,293122342,MDEyOklzc3VlQ29tbWVudDI5MzEyMjM0Mg==,11369233,2017-04-11T01:18:52Z,2017-04-11T01:18:52Z,MEMBER,"To reproduce: Connect two or more clients to same server teleport both to same location outdoors teleport one away to another outdoor location client disappears return client to original outdoor location @mp both clients from mp, teleport to another dungeon @teleloc 013A01E2 21.68 -50.02 6.11 -0.70 0.00 0.00 -0.71 (for example) the client that is supposed to portal away is left on the remaining client screen.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220811945 https://github.com/ACEmulator/ACE/issues/232#issuecomment-293149021,https://api.github.com/repos/ACEmulator/ACE/issues/232,293149021,MDEyOklzc3VlQ29tbWVudDI5MzE0OTAyMQ==,11369233,2017-04-11T04:49:50Z,2017-04-11T04:49:59Z,MEMBER,"Your location is: 0x2B110028 [104.277939 191.119308 48.005001] 0.221541 0.000000 0.000000 -0.975151 Your location is: 0x2B120021 [103.705467 0.314215 48.005001] 0.221541 0.000000 0.000000 -0.975151 locations of interest","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220838658 https://github.com/ACEmulator/ACE/pull/227#issuecomment-293284237,https://api.github.com/repos/ACEmulator/ACE/issues/227,293284237,MDEyOklzc3VlQ29tbWVudDI5MzI4NDIzNw==,25351661,2017-04-11T14:38:41Z,2017-04-11T14:38:41Z,CONTRIBUTOR,changelog...,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220749461 https://github.com/ACEmulator/ACE/issues/233#issuecomment-293324955,https://api.github.com/repos/ACEmulator/ACE/issues/233,293324955,MDEyOklzc3VlQ29tbWVudDI5MzMyNDk1NQ==,11369233,2017-04-11T16:48:24Z,2017-04-11T16:48:24Z,MEMBER,"I think this is a symptom of a lack of broadcasting of several things related to movement.. My guess it to resolve it queues will need to be built out for MoveToState and likely a change for UpdatePosition... ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220951042 https://github.com/ACEmulator/ACE/issues/233#issuecomment-293329824,https://api.github.com/repos/ACEmulator/ACE/issues/233,293329824,MDEyOklzc3VlQ29tbWVudDI5MzMyOTgyNA==,11369233,2017-04-11T17:04:38Z,2017-04-11T17:04:38Z,MEMBER,AutonomousPosition might be another event needing a queue as well to help smooth things out,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220951042 https://github.com/ACEmulator/ACE/pull/227#issuecomment-293478980,https://api.github.com/repos/ACEmulator/ACE/issues/227,293478980,MDEyOklzc3VlQ29tbWVudDI5MzQ3ODk4MA==,26508216,2017-04-12T05:38:47Z,2017-04-12T05:38:47Z,CONTRIBUTOR,Fixed changelog.md,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220749461 https://github.com/ACEmulator/ACE/pull/235#issuecomment-293739770,https://api.github.com/repos/ACEmulator/ACE/issues/235,293739770,MDEyOklzc3VlQ29tbWVudDI5MzczOTc3MA==,37453,2017-04-12T23:58:29Z,2017-04-12T23:58:29Z,CONTRIBUTOR,Just pulled the latest changes and tested again; all functions were working as expected.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220965499 https://github.com/ACEmulator/ACE/pull/229#issuecomment-293753308,https://api.github.com/repos/ACEmulator/ACE/issues/229,293753308,MDEyOklzc3VlQ29tbWVudDI5Mzc1MzMwOA==,25460553,2017-04-13T01:40:46Z,2017-04-13T01:40:46Z,CONTRIBUTOR,Updated and ready for merge upon approval.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220806148 https://github.com/ACEmulator/ACE/pull/236#issuecomment-293918593,https://api.github.com/repos/ACEmulator/ACE/issues/236,293918593,MDEyOklzc3VlQ29tbWVudDI5MzkxODU5Mw==,12808515,2017-04-13T14:52:58Z,2017-04-13T14:53:45Z,CONTRIBUTOR,"I have a thought on this, but will require some refactor work. I think we need to move all GameMessage/GameEvents to store inputted data, and then we add a byte[] GetPayload(Session session) function which actually writes the content for sending. Then a single event can be used repeatedly to broadcast to many sessions. I need to take a closer look at this to think out the details.","{""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/237#issuecomment-294036680,https://api.github.com/repos/ACEmulator/ACE/issues/237,294036680,MDEyOklzc3VlQ29tbWVudDI5NDAzNjY4MA==,26508216,2017-04-13T22:20:45Z,2017-04-13T22:20:45Z,CONTRIBUTOR,"As discussed with fantoms, I'll redo this as a proper commit.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,221690158 https://github.com/ACEmulator/ACE/pull/229#issuecomment-294223013,https://api.github.com/repos/ACEmulator/ACE/issues/229,294223013,MDEyOklzc3VlQ29tbWVudDI5NDIyMzAxMw==,25460553,2017-04-14T19:47:35Z,2017-04-14T19:47:35Z,CONTRIBUTOR,Stackoverflow pointed out a part of code that was not checking for existence prior to usage - I made that fix.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,220806148 https://github.com/ACEmulator/ACE/pull/238#issuecomment-294299000,https://api.github.com/repos/ACEmulator/ACE/issues/238,294299000,MDEyOklzc3VlQ29tbWVudDI5NDI5OTAwMA==,26508216,2017-04-15T15:09:00Z,2017-04-15T15:09:00Z,CONTRIBUTOR,I fixed and improved this according to your suggestions and will submit a new PR. Thanks all for the help!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,221699510 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 https://github.com/ACEmulator/ACE/issues/254#issuecomment-295032015,https://api.github.com/repos/ACEmulator/ACE/issues/254,295032015,MDEyOklzc3VlQ29tbWVudDI5NTAzMjAxNQ==,1765994,2017-04-19T01:10:33Z,2017-04-19T01:10:33Z,CONTRIBUTOR,"Some info on the Natural Regen rates can be seen at: http://acpedia.org/wiki/Natural_Resistances#Natural_Resistance_Bonus Mostly this references the modifiers to the natural rates due to attributes (and spells). I'm not sure where to find the actual natural rate. Perhaps some math can be done on some pcaps showing vital restoration and we can work back from the characters stats and spells? ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,222587429 https://github.com/ACEmulator/ACE/issues/254#issuecomment-295038094,https://api.github.com/repos/ACEmulator/ACE/issues/254,295038094,MDEyOklzc3VlQ29tbWVudDI5NTAzODA5NA==,1765994,2017-04-19T01:46:26Z,2017-04-19T01:46:26Z,CONTRIBUTOR,"I can't find the official natural regen formula documented. Following is from a Letter to the Players, Nov 2001. > b. New regeneration rates--As the game of Asheron's Call grows and progresses, certain design dynamics have become outdated. The hit point regeneration rate has become one of those antiquated systems. We wanted to decrease downtime, especially the downtime for those without life magic who also don't have healing. We think we've struck the right compromise with the current hit point and Stamina regen rates. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,222587429