INN / umbrella-energynewsnetwork

Umbrella repository for the Energy News Network site, a project of Fresh Energy
https://energynews.us
GNU General Public License v2.0
0 stars 1 forks source link

Digest search functionality #83

Closed benlk closed 4 years ago

benlk commented 4 years ago

Changes

This pull request makes the following changes:

Form in Firefox: Screen Shot 2020-02-10 at 13 17 13

Firefox's datepicker Screen Shot 2020-02-10 at 13 27 24

Inputting information in Safari, which lacks datepicker support (https://bugs.webkit.org/show_bug.cgi?id=119175 and https://caniuse.com/#feat=input-datetime) Screen Shot 2020-02-10 at 13 24 37 Screen Shot 2020-02-10 at 13 27 12

Why

For #77

Testing/Questions

Features that this PR affects:

Questions that need to be answered before merging:

Steps to test this PR:

  1. check out the PR, using a recent DB from production
  2. go to /category/digests/
  3. perform various searches:
    • by word
    • by date
    • by region
    • by any combination thereof
  4. Examine each search's results to determine whether the returned posts match the query
joshdarby commented 4 years ago

@benlk Where should results for the search be appearing? This is what I see after each search I've tried

Screen Shot 2020-02-12 at 3 47 24 PM

joshdarby commented 4 years ago

@benlk mentioned that I needed to actually set each post in the appropriate taxonomy instead of post category, which ended up working.