cityofaustin / atd-data-tech

Austin Transportation Data & Technology Services
17 stars 2 forks source link

Integrate authoritative employee data from Banner #2167

Closed johnclary closed 3 years ago

johnclary commented 4 years ago

We're currently keeping employee records up to date manually via import and/or cross referencing other sources. We should pull authoritative employee data into the app, e.g:

This data can come from Banner and/or Active Directory

johnclary commented 4 years ago

Not required for MVP. I reached out to both HRD (for Banner) and CTM (for Active Directory) to understand our options. Will report back. FYI @JaceDeloney

JaceDeloney commented 4 years ago

@johnclary do any of these data sources include an "essential" designation? Asking re #2170

johnclary commented 4 years ago

negative. the essential designation is being determined on an ad-hoc basis by staff. will comment on that issue.

johnclary commented 4 years ago

we have API access from HRD! i've added that info to 1password. next step is to integrate the appropriate data from that API into the system. leaving in backlog for now.

johnclary commented 3 years ago

Making progress in this repo. Reached out to Nathan today with a problem:

Nathan—I've been working to set up the integration between the Banner system and our Knack HR app. This will ensure that anyone who needs to submit their NEO forms, telework request, etc. will have access to the system automatically.

I've run across an issue where it seems that employees' email addresses in Banner are not current. A few employees have a missing email address, and others' email addresses in Banner are different than what's in Outlook.

So I'm assuming the situation is that (1) sometimes the employee is created in Banner before they're assigned an email address and (2) there's no way for y'all who maintain Banner to get notified when CTM makes a change to someone's email address. Is that right?

In the near term, I think we'll just have to let you know when we run into this problem, and ask you to update Banner. In the longer term I will reach out to CTM to see if they can provide us the data we need to keep things square.

johnclary commented 3 years ago

Todo:

amenity commented 3 years ago

@johnclary - Sprint 39?

amenity commented 3 years ago

@chiaberry - eying this for you. @johnclary said it will need to be better scoped and is probably a two-weeker.

amenity commented 3 years ago

Use case for this integration: Power users who manage applications need to be notified when an employee is separated. E.g. Staff from other divisions who have accounts in Signs and Markings' GIS need access revoked.

johnclary commented 3 years ago
moorerst commented 3 years ago

@chiaberry Per CTM the script to export employee data out of Active Directory is run once a month but can be requested more often.

moorerst commented 3 years ago

@chiaberry we had a brief discussion with HR about the Banner Integration project being active. Here are a few comments they had about what is important for them:

chiaberry commented 3 years ago

@moorerst thank you for the updates!

chiaberry commented 3 years ago

I count 8 employees in the email spreadsheet in the gdrive that don't have employee IDs associated with their names and emails, a few are in banner (with employee ids), but I found one so far that is in the CTM list and not in banner.

Is it safe to ignore emails in the csv without employeeIDs?

dianamartin commented 3 years ago

@chiaberry I think it's safe to do that because perhaps the employee ID will be updated in BANNER and come over in a different update.

This morning I did notice some accounts were "inactive". Did you run an update by chance? It was for the test accounts, I had to just re-add the passwords from 1password.

chiaberry commented 3 years ago

I inadvertently ran an update to the production app, I fixed my environment so it won't happen again.

Should Test Accounts never be marked as "inactive"?

chiaberry commented 3 years ago

Or is it that the Test Accounts got their passwords overwritten?

dianamartin commented 3 years ago

@chiaberry It's a little bit of both. We never normally inactive Test accounts bc they're always used/shared with the team in our password manager. I think also bc their passwords got wiped, the password managers passwords didn't work. So it was an easy enough fix to update them in Knack, but that was when I realized that could be the issue. Not sure if it's possible to keep them outside of the updates.

chiaberry commented 3 years ago

PR ready for review: https://github.com/cityofaustin/atd-knack-banner/pull/1

@moorerst & @dianamartin How frequent do you want the script to run? And is there a preferred day and time?

Can we request a consistent filename from CTM for the email address csv? Instead of 2021-07-12-ATDUsers.csv, ATDUsers.csv perhaps?

dianamartin commented 3 years ago

@chiaberry @moorerst Nathan said it can run as often as the Banners system updates. So I think every pay period (ex. 9/17, 10/3)

moorerst commented 3 years ago

@chiaberry @dianamartin - so you want the consistent filename ATDUsers.csv with no dates. So the filename will stay the same and be overwritten bi weekly?

chiaberry commented 3 years ago

@moorerst @dianamartin @johnclary ideally yes. They are free to upload a version of the csv with the date in the filename, along with overwriting the ATDUsers.csv.

Otherwise I'll need to update the code to figure out how to handle the filenames.

moorerst commented 3 years ago

@chiaberry I'll make the request to CTM. Thanks

moorerst commented 3 years ago

@chiaberry @dianamartin CTM wants to know what day/time you want the Active Directory employee data exported and starting on what date. i.e. every other Sunday @ midnight???

moorerst commented 3 years ago

@chiaberry Diana and I have a meeting with HR later today to talk about their HR forms Go Live but they always want to know how the Banner integration is going. Do you have an update I can provide them? Thanks

chiaberry commented 3 years ago

update: script is done and has been run, but I have not turned on the scheduled run yet.

Friday I realized that you @moorerst are not included in the list of employees in Banner, I am guessing it only includes full time employees? This script will mark anyone not in Banner as "inactive", will it be an issue if part time employees are marked as inactive in the HR app?

As for the previous question about CTM, could they do Friday night or Saturday night? I am scheduling our script to update at 5am on sundays.

johnclary commented 3 years ago

@chiaberry i know you already talked to Diana about this. Stephanie is a contractor now. If you want to double-check a part-time employee you can verify that Tilly is not deactivated.