Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Emergency Contacts display should filter to current accounts #30

Open
inghamn opened this issue Dec 4, 2018 · 1 comment
Open

Emergency Contacts display should filter to current accounts #30

inghamn opened this issue Dec 4, 2018 · 1 comment

Comments

@inghamn
Copy link
Member

inghamn commented Dec 4, 2018

The CSV version of the contacts list filters out people who are no longer in ActiveDirectory. The contact info for those people is still stored in the local mysql database; we just don't display their row.

We should probably have the HTML display for Emergency Contacts also filter out old user accounts, to match the CSV version.

Do we also want to do some cleanup; and have directory automatically purge contact info for people who are no longer in AD?

@charlesbrandt
Copy link

The only use that I could imagine for the old user data is to identify which accounts need to be removed from the Emergency Contact system itself. However, if we wipe everything and bulk import to the Emergency Contact system on every update, then that also solves the issue. In that case, I can't really see a reason to keep inactive accounts in the local database.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants