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

Digest page: Testing & QA #82

Open
8 of 11 tasks
MirandaEcho opened this issue Feb 3, 2020 · 6 comments
Open
8 of 11 tasks

Digest page: Testing & QA #82

MirandaEcho opened this issue Feb 3, 2020 · 6 comments
Milestone

Comments

@MirandaEcho
Copy link
Collaborator

MirandaEcho commented Feb 3, 2020

  • Add outline to submit feedback widget to match the others
  • Remove the old signup form
  • "Manage Subscription" text should match other button text
  • Sign up form: Email field should be required, all lists checked by default
  • All fonts in widgets, buttons should match site fonts, not the wireframe fonts
  • Can we make the checkboxes & lists text larger?
  • Feed: Needs to be set to shorter excerpts
  • 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.
@MirandaEcho MirandaEcho added this to the FRES-011 milestone Feb 3, 2020
@benlk
Copy link
Collaborator

benlk commented Mar 11, 2020

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

@joshdarby
Copy link

joshdarby commented Mar 11, 2020

Sign up form: Email field should be required

@MirandaEcho The email field is already required.

@joshdarby
Copy link

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
Copy link
Collaborator

benlk commented Mar 11, 2020

  • 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
Copy link
Collaborator Author

@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
Copy link
Collaborator

benlk commented Mar 11, 2020

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.

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

No branches or pull requests

3 participants