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

Fixes #1224: Show all posts for an author sorted by date when searching by Author #1256

Merged
merged 3 commits into from
Nov 15, 2020

Conversation

joelazwar
Copy link
Contributor

Issue This PR Addresses

Issue #1224

Type of Change

  • Bugfix: Change which fixes an issue
  • New Feature: Change which adds functionality
  • Documentation Update: Change which improves documentation
  • UI: Change which improves UI

Description

Changes search by author function to display the initial result (container that links to feed) and the addition of the timeline of their posts
joelauthorsearch

works with with multiple persons with shared names, but is limited to max posts per page. For best results, search for author's full name.

Checklist

  • Quality: This PR builds and passes our npm test and works locally
  • Tests: This PR includes thorough tests or an explanation of why it does not
  • Screenshots: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not (if applicable)
  • Documentation: This PR includes updated/added documentation to user exposed functionality or configuration variables are added/changed or an explanation of why it does not(if applicable)

Copy link
Contributor

@humphd humphd left a comment

Choose a reason for hiding this comment

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

This is really interesting. I love being able to get the posts for a user. I'm not 100% what to think about mixing the Author "card" with the results, whether it's a good thing or not.

One thing, when we fix #1260 this will maybe bee moot, since we can always show results for a specific author.

src/frontend/src/components/SearchPage/SearchPage.jsx Outdated Show resolved Hide resolved
@humphd
Copy link
Contributor

humphd commented Nov 14, 2020

@joelazwar Where does this stand? At the very least we need a rebase to get the SearchPage changes in.

@joelazwar
Copy link
Contributor Author

@humphd Right, I haven't had the time to think of a new design lately, so I'll just stick with the timeline of posts as a result. I'll go ahead and rebase this now

@joelazwar
Copy link
Contributor Author

joelazwar commented Nov 15, 2020

I rebased my branch but its practically more like a new branch I guess, since I undid all the changes (#1295 basically fixed the functionality) but I edited the query so that searched posts timeline is now sorted by date, however this is true for both author and posts filters. Let me know if you want to change that

src/backend/utils/indexer.js Outdated Show resolved Hide resolved
Copy link
Contributor

@humphd humphd left a comment

Choose a reason for hiding this comment

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

So this issue is fixing "Show all posts for an author sorted by date when searching by Author" but you seem to be doing it for all filter types. Can we limit this to filter='author'?

src/backend/utils/indexer.js Outdated Show resolved Hide resolved
@joelazwar
Copy link
Contributor Author

@humphd done 189961e

src/backend/utils/indexer.js Outdated Show resolved Hide resolved
src/backend/utils/indexer.js Outdated Show resolved Hide resolved
@humphd humphd requested a review from c3ho November 15, 2020 19:39
@c3ho
Copy link
Contributor

c3ho commented Nov 15, 2020

Searching by author doesn't work for me, did a search using incognito on Chrome, no results. Post worked though

@humphd humphd merged commit e45b521 into Seneca-CDOT:master Nov 15, 2020
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.

Show all posts for an author sorted by date when searching by Author
5 participants