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

Fix try fec.gov btn on legal no-results page #3716

Merged
merged 6 commits into from
May 4, 2020

Conversation

johnnyporkchops
Copy link
Contributor

@johnnyporkchops johnnyporkchops commented Apr 26, 2020

Summary

Point Try FEC.gov link on legal search no-results page to fec.gov search, passing the search query

Impacted areas of the application

modified: legal/templates/macros/legal.jinja
modified: fec/static/js/legal/SearchResults.js
modified: legal/templates/layouts/legal-doc-search-results.jinja

Screenshots

Apr-25-2020 23-49-47

How to test

checkout feature/fix-try-fec-link-legal-srch

Runserver:


@johnnyporkchops johnnyporkchops changed the title change link to fec search Fix link to fec,gov search on legal no-results page Apr 26, 2020
@johnnyporkchops johnnyporkchops changed the title Fix link to fec,gov search on legal no-results page Fix link to fec.gov search on legal no-results page Apr 26, 2020
@johnnyporkchops johnnyporkchops changed the title Fix link to fec.gov search on legal no-results page Fix try fec.gov search on legal no-results page Apr 26, 2020
@johnnyporkchops johnnyporkchops changed the title Fix try fec.gov search on legal no-results page Fix try fec.gov link on legal no-results page Apr 26, 2020
@johnnyporkchops johnnyporkchops changed the title Fix try fec.gov link on legal no-results page Fix try fec.gov btn on legal no-results page Apr 26, 2020
@johnnyporkchops johnnyporkchops added this to the Sprint 12.2 milestone Apr 27, 2020
@lbeaufort lbeaufort self-requested a review April 27, 2020 16:35
@lbeaufort lbeaufort removed their assignment Apr 27, 2020
Copy link
Member

@lbeaufort lbeaufort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@johnnyporkchops
Copy link
Contributor Author

johnnyporkchops commented May 1, 2020

@lbeaufort , I changed the link to global search in the other two places.

  • This link only shows when one types a keyword search.
  • For the legal-landing global searches, it always shows because this is essentially a one-field keyword search.
  • For the AO datatable that uses searchResults.js, I had to remove the part that echos the search term in the error message because, since it's react, it would update in place as a user types a NEW search term--essentially printing a message to the screen saying "We found no documents matching "the new search term(s)" before the search is performed, regardless of whether the NEW term returns results. This would be confusing to the user.

Copy link
Member

@lbeaufort lbeaufort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, thanks for helping me review!

@lbeaufort lbeaufort merged commit 06a8206 into develop May 4, 2020
@lbeaufort lbeaufort deleted the feature/fix-try-fec-link-legal-srch branch June 2, 2020 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix "try fec.gov" link on legal search "no results" message
2 participants