issue_comments: 504717494
This data as json
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/2033#issuecomment-504717494 | https://api.github.com/repos/ACEmulator/ACE/issues/2033 | 504717494 | MDEyOklzc3VlQ29tbWVudDUwNDcxNzQ5NA== | 8909245 | 2019-06-23T03:57:17Z | 2019-06-23T03:57:17Z | COLLABORATOR | ah i see some preliminary work for adding other vital types was in there, but the initial check was still always for Health This looks like a good change, and is moving in the right direction for all of the different healing kits types. I noticed a few other details for stamina/mana/unlimited use kits, mostly related to messages sent to the player. I can help clean these other messages up after this PR, unless anyone else would like to do that part Looks good to me, thanks again | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | 459532606 |