issue_comments: 485782183
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/1785#issuecomment-485782183 | https://api.github.com/repos/ACEmulator/ACE/issues/1785 | 485782183 | MDEyOklzc3VlQ29tbWVudDQ4NTc4MjE4Mw== | 3317737 | 2019-04-23T12:30:17Z | 2019-04-23T12:36:05Z | COLLABORATOR | > > > I mentioned this in DM's, but you can make it a lot easier for us to review this PR by creating a dummy Session.Network object that simply relays EnqueueSend down to the base session. > > It would leave the vast majority of these files unchanged so we can review the actual changes more clearly. > > Then, after the architecture change is approved and merged, the wrapper class can be removed. I'll see if it can't revert the reduction and do it right. Good call! | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | 435988213 |