-
Notifications
You must be signed in to change notification settings - Fork 41
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
Comments
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:
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: 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.
What's your take, @emileighoutlaw ? |
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: But it's possible I've oversimplified, so this idea would greatly benefit from someone who understands the term messiness better than I do |
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 My vote would be to keep the start and end date fields and add a simple plain text field for additional details. |
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. |
I think Jen's solution works. |
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: |
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:
But in reality there are a few exceptions to that template:
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
The text was updated successfully, but these errors were encountered: