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/1733#issuecomment-491626115,https://api.github.com/repos/ACEmulator/ACE/issues/1733,491626115,MDEyOklzc3VlQ29tbWVudDQ5MTYyNjExNQ==,26235642,2019-05-12T20:20:42Z,2019-05-12T20:20:42Z,CONTRIBUTOR,fairly certain the drudges (cats) jumped down in bsd,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,433055507
https://github.com/ACEmulator/ACE/issues/1723#issuecomment-491626352,https://api.github.com/repos/ACEmulator/ACE/issues/1723,491626352,MDEyOklzc3VlQ29tbWVudDQ5MTYyNjM1Mg==,26235642,2019-05-12T20:23:57Z,2019-05-12T20:23:57Z,CONTRIBUTOR,yes housing would refund pyreal stacks but not trade notes,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,432866170
https://github.com/ACEmulator/ACE/pull/1916#issuecomment-494574107,https://api.github.com/repos/ACEmulator/ACE/issues/1916,494574107,MDEyOklzc3VlQ29tbWVudDQ5NDU3NDEwNw==,26235642,2019-05-21T21:56:29Z,2019-05-21T21:56:29Z,CONTRIBUTOR,will move to feature branch,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,445979776
https://github.com/ACEmulator/ACE/pull/1924#issuecomment-497280727,https://api.github.com/repos/ACEmulator/ACE/issues/1924,497280727,MDEyOklzc3VlQ29tbWVudDQ5NzI4MDcyNw==,26235642,2019-05-30T10:16:22Z,2019-05-30T10:16:22Z,CONTRIBUTOR,any idea what the retail command did specifically?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,446858263
https://github.com/ACEmulator/ACE/pull/1948#issuecomment-497392257,https://api.github.com/repos/ACEmulator/ACE/issues/1948,497392257,MDEyOklzc3VlQ29tbWVudDQ5NzM5MjI1Nw==,26235642,2019-05-30T16:27:28Z,2019-05-30T16:27:28Z,CONTRIBUTOR,"was wondering if this was the right spot, thanks","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,450231941
https://github.com/ACEmulator/ACE/pull/1924#issuecomment-500160713,https://api.github.com/repos/ACEmulator/ACE/issues/1924,500160713,MDEyOklzc3VlQ29tbWVudDUwMDE2MDcxMw==,26235642,2019-06-08T20:32:37Z,2019-06-08T20:32:37Z,CONTRIBUTOR,"this is definitely more fun, and serves the same purpose.

i can not figure out why i need two calls to setmaxvitals() to get the hp to update in the client. it was working back when i was first looking at this, now it needs the second call. stam and mana seem to update just fine with the first call. also it appears to only be the client, calling /showstats reports the correct values in the terminal for health - just not in the client.

ultimately not a huge deal, just leave the second call to setmaxvitals in there... but it may be something that affects other areas so worth noting","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,446858263
https://github.com/ACEmulator/ACE/pull/1984#issuecomment-500308633,https://api.github.com/repos/ACEmulator/ACE/issues/1984,500308633,MDEyOklzc3VlQ29tbWVudDUwMDMwODYzMw==,26235642,2019-06-10T06:37:46Z,2019-06-10T06:37:46Z,CONTRIBUTOR,So turns out I am mistaken and in retail if you deleted a character you were able to create another character with the same name immediately and get an error when trying to restore if someone has taken that name.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,454016320
https://github.com/ACEmulator/ACE/pull/2000#issuecomment-501986121,https://api.github.com/repos/ACEmulator/ACE/issues/2000,501986121,MDEyOklzc3VlQ29tbWVudDUwMTk4NjEyMQ==,26235642,2019-06-14T06:30:05Z,2019-06-14T06:30:05Z,CONTRIBUTOR,Doesn't it also need to be applied to missile weapons? Is that taken care of somewhere else?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,456071830
https://github.com/ACEmulator/ACE/pull/2000#issuecomment-501999214,https://api.github.com/repos/ACEmulator/ACE/issues/2000,501999214,MDEyOklzc3VlQ29tbWVudDUwMTk5OTIxNA==,26235642,2019-06-14T07:21:48Z,2019-06-14T07:21:48Z,CONTRIBUTOR,missile wep speed is used for the reload animation,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,456071830
https://github.com/ACEmulator/ACE/pull/2027#issuecomment-504677359,https://api.github.com/repos/ACEmulator/ACE/issues/2027,504677359,MDEyOklzc3VlQ29tbWVudDUwNDY3NzM1OQ==,26235642,2019-06-22T15:59:19Z,2019-06-22T15:59:19Z,CONTRIBUTOR,what happens if the server crashes before you /ungod? ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,459416589
https://github.com/ACEmulator/ACE/pull/2027#issuecomment-504805622,https://api.github.com/repos/ACEmulator/ACE/issues/2027,504805622,MDEyOklzc3VlQ29tbWVudDUwNDgwNTYyMg==,26235642,2019-06-24T01:04:48Z,2019-06-24T01:04:48Z,CONTRIBUTOR,"When you say PropertyString do you mean creating a new entry in the db? Or do you mean serializing the data and saving it in something that is already existing? Is there an example of something that is currently saved like this?

That could also work but isn't sort of the same thing - just on a different (user controllable) timescale? The way it is setup now it forces the reload on log out (or total reload if the server crashes) which should create shorter 'unsaved' time periods. /ungod would just force the reload when the user chooses.

I appreciate the feedback. I'm open to implementing this however we agree is best - or totally dropping it if we agree it is useless, or just too dangerous.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,459416589
https://github.com/ACEmulator/ACE/pull/2027#issuecomment-507500810,https://api.github.com/repos/ACEmulator/ACE/issues/2027,507500810,MDEyOklzc3VlQ29tbWVudDUwNzUwMDgxMA==,26235642,2019-07-02T03:11:52Z,2019-07-02T03:14:13Z,CONTRIBUTOR,"i liked gm's approach so i ran with it

i think this now gives the appropriate level of functionality without the danger of not saving to the db

it could even act as a crude way to restore character templates with some tweaks","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,459416589
https://github.com/ACEmulator/ACE/issues/2070#issuecomment-508295236,https://api.github.com/repos/ACEmulator/ACE/issues/2070,508295236,MDEyOklzc3VlQ29tbWVudDUwODI5NTIzNg==,26235642,2019-07-04T00:21:29Z,2019-07-04T00:21:29Z,CONTRIBUTOR,"Two possibilities I tested.

- hit by a void blast as a npk
- hit by a two hander as a npk

both seemed to have no effect as I was able to recall right after and did not get stuck.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,463984016
https://github.com/ACEmulator/ACE/pull/2027#issuecomment-509895540,https://api.github.com/repos/ACEmulator/ACE/issues/2027,509895540,MDEyOklzc3VlQ29tbWVudDUwOTg5NTU0MA==,26235642,2019-07-10T03:44:49Z,2019-07-10T03:44:49Z,CONTRIBUTOR,anything else this needs?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,459416589
https://github.com/ACEmulator/ACE/pull/2107#issuecomment-511247043,https://api.github.com/repos/ACEmulator/ACE/issues/2107,511247043,MDEyOklzc3VlQ29tbWVudDUxMTI0NzA0Mw==,26235642,2019-07-15T00:16:20Z,2019-07-15T00:16:20Z,CONTRIBUTOR,thanks Theran,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,467893739
https://github.com/ACEmulator/ACE/issues/2070#issuecomment-514004535,https://api.github.com/repos/ACEmulator/ACE/issues/2070,514004535,MDEyOklzc3VlQ29tbWVudDUxNDAwNDUzNQ==,26235642,2019-07-23T00:26:09Z,2019-07-23T00:26:09Z,CONTRIBUTOR,Not sure what exactly fixed it but it hasn't happened in a while. I'll close it.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,463984016
https://github.com/ACEmulator/ACE/pull/2388#issuecomment-550129208,https://api.github.com/repos/ACEmulator/ACE/issues/2388,550129208,MDEyOklzc3VlQ29tbWVudDU1MDEyOTIwOA==,26235642,2019-11-06T03:41:51Z,2019-11-06T03:41:51Z,CONTRIBUTOR,"figured i would fix the root of the problem too by checking against the valid skills hashset before adding new skills with GetOrAddSkill

not sure if this fix is appropriate... based on my understanding i think this would still return a creature skill object if the call is made with an inappropriate skill value, but it wouldnt add the value to the db. the onus is on the user of the function to make sure they arent checking or using invalid skills at that point... with 99+ references to GetCreatureSkill i cant possibly check them all","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,517525522
https://github.com/ACEmulator/ACE/pull/2388#issuecomment-550414587,https://api.github.com/repos/ACEmulator/ACE/issues/2388,550414587,MDEyOklzc3VlQ29tbWVudDU1MDQxNDU4Nw==,26235642,2019-11-06T17:25:39Z,2019-11-06T17:25:39Z,CONTRIBUTOR,"Ah. Definitely didn’t realize that mobs used deprecated skills.

I’ll back these changes out of this PR when I get home and possibly look into fixing this elsewhere in the call chain like suggested. 

Thanks guys.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,517525522