bcgov / ExitSurveyAdmin

The Exit Survey Administration Tool will assist BC Stats in administering the BCPS Exit Survey.
https://www2.gov.bc.ca/gov/content/data/statistics/government/employee-research/exit-survey
Apache License 2.0
3 stars 1 forks source link

Bring over BC Assessment LDAP lookup case code from NJSA #203

Closed hangler closed 1 year ago

hangler commented 1 year ago

@aclowery For clarity, in ESA, do we only want to use this for setting the email, or do we also want to update other fields (first name, last name, location city) similarly to how we do in NJSA?

aclowery commented 1 year ago

Sure, let's try it how NJSA does with the additional fields from LDAP added to each record (if not in LDAP which is the case for many I believe, use what's in the API). Will there be any issues when we port over the PROD data and they don't have these LDAP variables in the data? Would it do LDAP lookups on them? Would they just default to blanks?

aclowery commented 1 year ago

Just realizing LDAP is the only way of getting emails for anyone as email address is not a variable in the CSV and likely not in the API either?

hangler commented 1 year ago

That's right, there's no concept of a "CHIPS" email as there is in NJSA.

hangler commented 1 year ago

@aclowery:

Sure, let's try it how NJSA does with the additional fields from LDAP added to each record (if not in LDAP which is the case for many I believe, use what's in the API). Will there be any issues when we port over the PROD data and they don't have these LDAP variables in the data? Would it do LDAP lookups on them? Would they just default to blanks?

Hope that sounds OK!

aclowery commented 1 year ago

I think that sounds fine!

hangler commented 1 year ago

We looked at this together and confirmed it was working.