GitHub PullRequestReviewCommentEvent at 2014-05-23 01:46:06
Repository: packetfence
Actor: cgx
Payload: {"comment":{"url":"https://api.github.com/repos/inverse-inc/packetfence/pulls/comments/12983250","id":12983250,"diff_hunk":"@@ -0,0 +1,101 @@\n+# Vlan filter configuration\n+# \n+# you can trigger rule on specific scope (NormalVlan, RegistrationVlan, ViolationVlan, AutoRegistration)\n+#\n+# Make a simple rule like this:\n+#\n+# [1]","path":"conf/vlan_filters.conf.example","position":7,"original_position":7,"commit_id":"c2e4cacdc72470b10763ad3ec66672372430ffa5","original_commit_id":"c2e4cacdc72470b10763ad3ec66672372430ffa5","user":{"login":"cgx","id":1088448,"avatar_url":"https://avatars.githubusercontent.com/u/1088448?","gravatar_id":"0dfa5f3ae397487697507e49d0470ac5","url":"https://api.github.com/users/cgx","html_url":"https://github.com/cgx","followers_url":"https://api.github.com/users/cgx/followers","following_url":"https://api.github.com/users/cgx/following{/other_user}","gists_url":"https://api.github.com/users/cgx/gists{/gist_id}","starred_url":"https://api.github.com/users/cgx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cgx/subscriptions","organizations_url":"https://api.github.com/users/cgx/orgs","repos_url":"https://api.github.com/users/cgx/repos","events_url":"https://api.github.com/users/cgx/events{/privacy}","received_events_url":"https://api.github.com/users/cgx/received_events","type":"User","site_admin":false},"body":"Can you use a significant name for the rule?","created_at":"2014-05-23T01:46:06Z","updated_at":"2014-05-23T01:46:06Z","html_url":"https://github.com/inverse-inc/packetfence/pull/133#discussion_r12983250","pull_request_url":"https://api.github.com/repos/inverse-inc/packetfence/pulls/133","_links":{"self":{"href":"https://api.github.com/repos/inverse-inc/packetfence/pulls/comments/12983250"},"html":{"href":"https://github.com/inverse-inc/packetfence/pull/133#discussion_r12983250"},"pull_request":{"href":"https://api.github.com/repos/inverse-inc/packetfence/pulls/133"}}}}