Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

[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