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

Update results display for MUR search #5255

Closed
1 task
Tracked by #221
JonellaCulmer opened this issue May 26, 2022 · 3 comments · Fixed by #5319
Closed
1 task
Tracked by #221

Update results display for MUR search #5255

JonellaCulmer opened this issue May 26, 2022 · 3 comments · Fixed by #5319
Assignees
Milestone

Comments

@JonellaCulmer
Copy link
Contributor

JonellaCulmer commented May 26, 2022

What are we after:
Update MUR search results display with election cycle, disposition, subject, document name, document URL, document type, removed ellipses, force all-caps for committee name

MUR search epic:
fecgov/fec-epics#221

Completion criteria:

  • Update results display from design

##Related pull-requests
#5175

Prototype for matching keyword interaction:
https://invis.io/YN12TP3Q2XQJ#/468022210_Mur_Main

Updated mockups with keyword matching included:
MUR-Document with lines

@patphongs
Copy link
Member

@JonellaCulmer A disposition can have multiple respondents. See example for MUR 7927.

Screen Shot 2022-06-23 at 8 17 03 PM

In the design, it appears dispositions are listed uniquely and don't necessarily repeat. Do you want the dispositions to be listed just once or do you want it to repeat?

Disposition(s): Conciliation-PPC; Conciliation-PPC; 

@patphongs patphongs self-assigned this Jun 24, 2022
@JonellaCulmer
Copy link
Contributor Author

@patphongs Dispositions should only be listed once. Are we able to include the number of respondents for each disposition? Ex. Conciliation-PPC (2 respondents)

@patphongs
Copy link
Member

Dispositions should only be listed once

Thanks @JonellaCulmer, I forgot that we've had this conversation already. I should be able to do that.

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

Successfully merging a pull request may close this issue.

3 participants