html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue
https://github.com/ACEmulator/ACE/pull/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