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

Add a "last employment date" field #604

Open
ssempervirens opened this issue Dec 17, 2018 · 1 comment · May be fixed by #642
Open

Add a "last employment date" field #604

ssempervirens opened this issue Dec 17, 2018 · 1 comment · May be fixed by #642
Assignees
Projects

Comments

@ssempervirens
Copy link
Member

When officers resign or quit, we should have a way of displaying that information. I think this could look like having two fields—last employment date field (which should default to N/A) and some sort of explanation/additional details field—i.e. if they resigned or were fired for some sort of misconduct issue.

@ghost ghost added this to In progress in 0.6 Jan 29, 2019
@ghost ghost moved this from In progress to To do in 0.6 Jan 29, 2019
@McEileen McEileen self-assigned this Feb 18, 2019
@McEileen McEileen moved this from To do to In progress in 0.6 Feb 18, 2019
@McEileen
Copy link
Collaborator

I'd appreciate some feedback on how to display this on individual officers' pages. My instinct is that we should only display the "last employment date field" if the officer is not currently employed. I also think we should only display the "explanation/additional details" field if it isn't blank. Thoughts?

@McEileen McEileen linked a pull request Mar 13, 2019 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
0.6
  
In progress
2 participants