Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix Search #1

Merged
merged 1 commit into from Jul 23, 2013

Conversation

Projects
None yet
2 participants
Contributor

jacobpatton commented Jul 23, 2013

This fixes the search feature, which is powered by
TipueSearch:
http://www.tipue.com/search/

The important part of this commit is that the
tipuesearch_content.erb script is now outputting
valid JSON. Before, it had errantly picked up the
search page and tried to add its javascript to the
JSON object.

Other changes:

  • Update the app to use the latest jQuery from
    Google's CDN
  • Update the TipueSearch lib to the latest version
  • Move the site's JS to its head -- no need for
    page-specific JS
@jacobpatton jacobpatton Fix Search
This fixes the search feature, which is powered by
TipueSearch:
http://www.tipue.com/search/

The important part of this commit is that the
tipuesearch_content.erb script is now outputting
valid JSON. Before, it had errantly picked up the
search page and tried to add its javascript to the
JSON object.

Other changes:

* Update the app to use the latest jQuery from
  Google's CDN
* Update the TipueSearch lib to the latest version
* Move the site's JS to its head -- no need for
  page-specific JS
0a15f30
Owner

alexknowshtml commented Jul 23, 2013

You, sir, are a king.

@alexknowshtml alexknowshtml added a commit that referenced this pull request Jul 23, 2013

@alexknowshtml alexknowshtml Merge pull request #1 from jacobpatton/jp/search
Fix Search
2aa2099

@alexknowshtml alexknowshtml merged commit 2aa2099 into alexknowshtml:master Jul 23, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment