...
[17:43:28 CST(-0600)] <JasonElwood> cool. thanks
[17:43:47 CST(-0600)] <dmccallum54> so if all you want to do is list relevant departments in the report form you should be able to query the person record (with a join, actually)
[17:44:10 CST(-0600)] <dmccallum54> if you then want to filter the coach list in the report form, i.e. client side, by department, you probably have to change the API
[17:44:19 CST(-0600)] <dmccallum54> b/c i bet that info isn't included
[17:44:29 CST(-0600)] <dmccallum54> (you're getting some sort of PersonLiteTO or CoachLiteTO)
[17:44:45 CST(-0600)] <JasonElwood> yeah. I'm just trying to make sure we don't have any holes in the department list
[17:45:13 CST(-0600)] <dmccallum54> actually … looks like CoachPersonLiteTO does have dept name in it.
[17:45:24 CST(-0600)] <dmccallum54> the department list won't be comprehensive
[17:45:31 CST(-0600)] <JasonElwood> Ideally we should carry over the department when the person is created so we aren't querying the external_person table. But I think it is working as designd.
[17:45:35 CST(-0600)] <dmccallum54> i.e. it won't include depts for which we have no current person records
[17:46:27 CST(-0600)] <dmccallum54> ah… you're saying that the persons created by these coach lookup operations aren't immediately synced with the entire external record?
[17:47:39 CST(-0600)] <JasonElwood> not about time. department doesn't exist in person
[17:48:07 CST(-0600)] <dmccallum54> right… it's on an associated object
[17:48:34 CST(-0600)] <dmccallum54> PersonStaffDetails
[17:48:45 CST(-0600)] <dmccallum54> think that was what jim was querying
[17:49:09 CST(-0600)] <JasonElwood> ugh. missed that one. sorry. the sync is working then
[17:49:13 CST(-0600)] <dmccallum54> yah
[17:49:14 CST(-0600)] <dmccallum54> https://github.com/Jasig/SSP/commit/2cd1f2acbb1ee45f908263b44e66db7211d70c49
[17:49:50 CST(-0600)] <dmccallum54> i did check the coach sync code that runs when we pull the coach list… it's exactly the same function that runs for the background sync. so the dept should be set immediately
[17:50:15 CST(-0600)] <JasonElwood> its as immediate as everything else. so it works fine. thanks
[17:51:01 CST(-0600)] <dmccallum54> heh