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

Commissioner serve dates are confusing #816

Closed
Tracked by #104
emileighoutlaw opened this issue Feb 13, 2017 · 6 comments
Closed
Tracked by #104

Commissioner serve dates are confusing #816

emileighoutlaw opened this issue Feb 13, 2017 · 6 comments

Comments

@emileighoutlaw
Copy link
Contributor

Recapping conversation that happened in FEC's Slack:

The "serve" dates in our Commissioner page Wagtail templates don't work for messy dates. This is how we've created the template:

screen shot 2017-02-13 at 2 19 33 pm

But in reality there are a few exceptions to that template:

  • Gaps in service without a reappointment
  • Recess appointments (which lead to things like multiple swearing in dates)
  • Commissioners serving on an expired term

Messy dates impact both current and former Commissioners, and we need to sort this out before those pages can go live.

@AmyKort and @PaulClark2 brought up that they'd support an approach with as little information in the first iteration as possible, since it might be difficult to shoehorn this into template.

cc @jenniferthibault @noahmanger

@jenniferthibault
Copy link
Contributor

I'm having a hard time knowing what the need is behind this, especially when looking at what happens on the current FEC site.

On the current FEC site, for current commissioners, I see this handled through description in the written text of the commissioner's bios:

Steven T. Walther was first sworn in as an FEC Commissioner on January 10, 2006, as a recess appointee. Although his name was placed before the Senate for confirmation in June, 2007, his recess term expired on December 31, 2007, before the Senate acted. On June 24, 2008, he was confirmed unanimously by the Senate and sworn in by Supreme Court Justice Ruth Bader Ginsburg on June 27, to resume the balance of his statutory term. Mr. Walther served as Vice Chairman of the FEC for the balance of 2008 and served as the Commission’s Chairman in 2009.

screen shot 2017-02-13 at 4 39 26 pm

For former commissioners, I see a lightweight intro to this on the index page, but a more detailed description also in the written text of the commissioner's bios:
screen shot 2017-02-13 at 4 40 48 pm

Given the complexity around reappointments & expired / still serving terms, I'm not sure there is an elegant design solution without the benefit of the plain language write-up in each commissioner's bio.

With this, I'm leaning that we don't make more custom fields that could generate confusing date ranges, since there is almost always an exception to the rule. Instead, we could have a generic field that accepts plain text entry, where we could essentially match what's on the current FEC site.

(Reappointed in July 1987, July 1994 and July 2000)

What's your take, @emileighoutlaw ?

@jenniferthibault
Copy link
Contributor

jenniferthibault commented Feb 13, 2017

Actually! The more simple solution for now might be to just type the word "Reappointed" in the field itself, and enter only the year someone was reappointed, not the full DD/MM/YYYY. That would allow us to keep the fields as they are, but customize language around anything tricky (which I admit I don't fully understand, except reappointments)

Using @patphongs' examples from Slack, that would give us:
screen shot 2017-02-13 at 5 00 50 pm

screen shot 2017-02-13 at 4 57 24 pm

But it's possible I've oversimplified, so this idea would greatly benefit from someone who understands the term messiness better than I do

@noahmanger
Copy link

With only a partial understanding of all the intricacies, @jenniferthibault 's solution seems like a good one. It's valuable to store the start and end dates as actual date values in the database, because when displaying a list of past commissioners, they're ordered by expiration date.

My vote would be to keep the start and end date fields and add a simple plain text field for additional details.

@emileighoutlaw
Copy link
Contributor Author

I also feel like I don't fully understand the intricacies of this— I'm wondering what @PaulClark2, @AmyKort and @patphongs think of Jen's solution.

@PaulClark2
Copy link
Contributor

I think Jen's solution works.

@noahmanger
Copy link

Sounds like we have agreement here. I added "Reappointed" to the existing pages with reappointment dates, so closing. But just pinging @patphongs and @johnnyporkchops that for future pages you need to add "Reappointed" in this field:
image

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

4 participants