{"action":"created","issue":{"url":"https://api.github.com/repos/inverse-inc/packetfence/issues/516","labels_url":"https://api.github.com/repos/inverse-inc/packetfence/issues/516/labels{/name}","comments_url":"https://api.github.com/repos/inverse-inc/packetfence/issues/516/comments","events_url":"https://api.github.com/repos/inverse-inc/packetfence/issues/516/events","html_url":"https://github.com/inverse-inc/packetfence/pull/516","id":71706563,"number":516,"title":"Show mandatory fields when you select them on the portal profile","user":{"login":"fdurand","id":1553962,"avatar_url":"https://avatars.githubusercontent.com/u/1553962?v=3","gravatar_id":"","url":"https://api.github.com/users/fdurand","html_url":"https://github.com/fdurand","followers_url":"https://api.github.com/users/fdurand/followers","following_url":"https://api.github.com/users/fdurand/following{/other_user}","gists_url":"https://api.github.com/users/fdurand/gists{/gist_id}","starred_url":"https://api.github.com/users/fdurand/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fdurand/subscriptions","organizations_url":"https://api.github.com/users/fdurand/orgs","repos_url":"https://api.github.com/users/fdurand/repos","events_url":"https://api.github.com/users/fdurand/events{/privacy}","received_events_url":"https://api.github.com/users/fdurand/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/inverse-inc/packetfence/labels/for%20review","name":"for review","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/inverse-inc/packetfence/milestones/4","html_url":"https://github.com/inverse-inc/packetfence/milestones/Packetfence-5.1","labels_url":"https://api.github.com/repos/inverse-inc/packetfence/milestones/4/labels","id":1067163,"number":4,"title":"Packetfence-5.1","description":"PR and issues targeted for inclusion in the next+1 release, i.e. 5.1. ","creator":{"login":"louismunro","id":193997,"avatar_url":"https://avatars.githubusercontent.com/u/193997?v=3","gravatar_id":"","url":"https://api.github.com/users/louismunro","html_url":"https://github.com/louismunro","followers_url":"https://api.github.com/users/louismunro/followers","following_url":"https://api.github.com/users/louismunro/following{/other_user}","gists_url":"https://api.github.com/users/louismunro/gists{/gist_id}","starred_url":"https://api.github.com/users/louismunro/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/louismunro/subscriptions","organizations_url":"https://api.github.com/users/louismunro/orgs","repos_url":"https://api.github.com/users/louismunro/repos","events_url":"https://api.github.com/users/louismunro/events{/privacy}","received_events_url":"https://api.github.com/users/louismunro/received_events","type":"User","site_admin":false},"open_issues":23,"closed_issues":3,"state":"open","created_at":"2015-04-14T16:01:14Z","updated_at":"2015-04-30T19:30:20Z","due_on":null,"closed_at":null},"comments":2,"created_at":"2015-04-28T21:25:40Z","updated_at":"2015-05-01T17:00:15Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/inverse-inc/packetfence/pulls/516","html_url":"https://github.com/inverse-inc/packetfence/pull/516","diff_url":"https://github.com/inverse-inc/packetfence/pull/516.diff","patch_url":"https://github.com/inverse-inc/packetfence/pull/516.patch"},"body":"Description\r\n---------------\r\nShow mandatory fields when they have been selected on the portal profile\r\nAdded an config parameter in authentication sources (LDAP, AD, Htpasswd, Kerberos,Radius) to make the parameter mandatory or no. (if you have a guest auth source per example)\r\nUpdate the person table with mandatory field.\r\n\r\nImpacts\r\n----------\r\nNo impacts\r\n\r\nDelete branch after merge\r\n-----------------------------------\r\nYES\r\n\r\n\r\nNEWS file entries\r\n-------\r\nEnhancements\r\n++++++++++++\r\n* Show mandatory fields on the captive portal and update the person table with them.\r\n"},"comment":{"url":"https://api.github.com/repos/inverse-inc/packetfence/issues/comments/98177882","html_url":"https://github.com/inverse-inc/packetfence/pull/516#issuecomment-98177882","issue_url":"https://api.github.com/repos/inverse-inc/packetfence/issues/516","id":98177882,"user":{"login":"dwlfrth","id":633234,"avatar_url":"https://avatars.githubusercontent.com/u/633234?v=3","gravatar_id":"","url":"https://api.github.com/users/dwlfrth","html_url":"https://github.com/dwlfrth","followers_url":"https://api.github.com/users/dwlfrth/followers","following_url":"https://api.github.com/users/dwlfrth/following{/other_user}","gists_url":"https://api.github.com/users/dwlfrth/gists{/gist_id}","starred_url":"https://api.github.com/users/dwlfrth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dwlfrth/subscriptions","organizations_url":"https://api.github.com/users/dwlfrth/orgs","repos_url":"https://api.github.com/users/dwlfrth/repos","events_url":"https://api.github.com/users/dwlfrth/events{/privacy}","received_events_url":"https://api.github.com/users/dwlfrth/received_events","type":"User","site_admin":false},"created_at":"2015-05-01T17:00:15Z","updated_at":"2015-05-01T17:00:15Z","body":"Should remove anything related to SQL (for the moment... we need to talk about this) because we can't edit that source (as I said, need to talk about this.)\r\nHint: lib/pf/Authentication/Source/SQLSource.pm"}}