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

Updated the language for no search results found #3829

Merged
merged 3 commits into from
Jun 10, 2020

Conversation

rfultz
Copy link
Contributor

@rfultz rfultz commented Jun 8, 2020

Summary

Impacted areas of the application

Changed the language on the search results page when nothing is found

Screenshots

Current:
image

Updated:
image

Related PRs

None

How to test

Not sure who should test so I added a few of you. Feel free to bail or swap. Dorothy, I figured you can approve with the screenshot and/or dev

  • Pull the branch
  • npm i
  • npm run build
  • /manage.py runserver
  • Search for gibberish
    • Should see the new message
  • Search for something legit
    • Should see results and not the no-results-found message

Copy link
Contributor

@dorothyyeager dorothyyeager left a comment

Choose a reason for hiding this comment

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

@rfultz The screenshot looks fine. When I tested in dev, I got the old message.

@lbeaufort
Copy link
Member

lbeaufort commented Jun 9, 2020

@dorothyyeager the changes won't show up on dev unless we merge the PR, the only way to test this is either locally or asking a developer to do a manual deploy to dev, but that will get overwritten as soon as a PR to develop gets merged.

<p>If you'd like to contact our team, we're available by <a href="mailto:{{ settings.WEBMANAGER_EMAIL }}">email</a> or <a href="https://github.com/fecgov/fec">GitHub</a>.</p>
<p>Think this was a mistake?<br>Try one of the other search tools or submit feedback</p>
<p>
<a href="mailto:{{ settings.WEBMANAGER_EMAIL }}" class="button--standard">Email our team</a>&nbsp;&nbsp; <a href="https://github.com/fecgov/fec" class="button--standard">File an issue</a></p>
Copy link
Member

Choose a reason for hiding this comment

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

Should "File an issue" link to https://github.com/fecgov/fec/issues/new or https://github.com/fecgov/fec/issues?

It looks like we're linking to either one of those in other places. We should probably keep all of them consistent. https://github.com/fecgov/fec/issues/new seems like it might be friendliest.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That wasn't specified so I left it as it was, but I've updated it to new issues

@codecov-commenter
Copy link

codecov-commenter commented Jun 9, 2020

Codecov Report

Merging #3829 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #3829   +/-   ##
========================================
  Coverage    75.25%   75.25%           
========================================
  Files          121      121           
  Lines         7352     7352           
  Branches       592      592           
========================================
  Hits          5533     5533           
  Misses        1819     1819           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7d956fc...bf777e6. Read the comment docs.

Copy link
Member

@patphongs patphongs left a comment

Choose a reason for hiding this comment

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

Looks good

@lbeaufort lbeaufort merged commit 6aa3088 into develop Jun 10, 2020
@lbeaufort lbeaufort deleted the feature/3619-updated-message-for-no-search-results branch June 16, 2020 14:56
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.

Implement "no results" message for global search
6 participants