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

Solution. #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Solution. #40

wants to merge 1 commit into from

Conversation

iamsellek
Copy link

  • Add pagination to the API
  • Hook the Load More button into an event listener for clicks
  • ToLower all of the string search stuff so that search is case insensitive
  • Only return each instance of a search query one time (so results that have multiple results within the 250 chars before or after limit return all of those results as one result)

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

Successfully merging this pull request may close these issues.

1 participant