Digest page: Testing & QA

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 page: Testing & QA #82

Open MirandaEcho opened 4 years ago

MirandaEcho commented 4 years ago

[x] Add outline to submit feedback widget to match the others
[x] Remove the old signup form
[x] "Manage Subscription" text should match other button text
[x] Sign up form: Email field should be required, all lists checked by default
[x] All fonts in widgets, buttons should match site fonts, not the wireframe fonts
[x] Can we make the checkboxes & lists text larger?
[x] Feed: Needs to be set to shorter excerpts
[x] Move "Go" search button to bottom or search fields, make it say "Search" instead

Issues with Search:

[ ] Further testing needed
[ ] US Energy News shows up when just Midwest is checked
[ ] Another example: If you search for just Northeast and Southeast, Western shows up.

benlk commented 4 years ago

@MirandaEcho do you have links to example search queries for the "US Energy news shows up when just Midwest is checked" case?

joshdarby commented 4 years ago

Sign up form: Email field should be required

@MirandaEcho The email field is already required.

joshdarby commented 4 years ago

Feed: Needs to be set to shorter excerpts

@MirandaEcho These are already set to be pretty short, but the formatting of these articles is what's throwing it off I think. Can we set custom excerpts in the excerpt field like they've been doing for every other post?

benlk commented 4 years ago

If you search for just Northeast and Southeast, Western shows up

Confirmed at https://usenergystage.wpengine.com/category/digest/?digest-search=&search+submit=&after=&before=&digest-search-region%5B%5D=northeast&digest-search-region%5B%5D=southeast : The post https://usenergystage.wpengine.com/digests/utah-legislature-set-to-tackle-climate-change/ showed up, with the "Western Energy News" top term.

The digest's post_class list shows: id="post-1669514" class="hnews item post-1669514 roundup type-roundup status-publish format-standard hentry category-western-energy-news region-west

This is an interesting bug.

Also appearing in those search results is the post https://usenergystage.wpengine.com/2020/01/23/southeast/the-n-c-utilities-commission-is-almost-fully-filled-with-cooper-appointees-what-will-that-mean-in-2020/ which has "North Carolina" set as a top term. It's a standard post in the "news" category without the "digest" category set.

MirandaEcho commented 4 years ago

@MirandaEcho do you have links to example search queries for the "US Energy news shows up when just Midwest is checked" case?

@benlk: https://usenergystage.wpengine.com/category/digest/?digest-search=utah&search+submit=&after=&before=&digest-search-region%5B%5D=midwest

benlk commented 4 years ago

https://usenergystage.wpengine.com/category/digest/?digest-search=utah&search+submit=&after=&before=&digest-search-region%5B%5D=midwest

Every digest in this search results page that has a top term of the "US Energy News" category has the "Midwest" region term set.

The search is operating as designed: it's returning results from the Midwest region.