Skip to content
This repository has been archived by the owner on Feb 7, 2023. It is now read-only.

Implemented the search using lunr.js #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

RishikeshDarandale
Copy link

Instead of calling feed.json on every page, used HTML 5 sessionStorage
to store actual data and index of lunr.js

Instead of calling feed.json on every page, used HTML 5 sessionStorage
to store actual data and index of lunr.js
@philwareham
Copy link

Hi @RishikeshDarandale - I use your handy lunr implementation, but it seems to have stopped working with lunr v2. Any chance of a fix? Cheers!

@RishikeshDarandale
Copy link
Author

Hi @philwareham - I will check and get back to you! Thanks.

@Vonfry
Copy link

Vonfry commented Apr 10, 2017

I have the same problem.

olivernn/lunr.js#254

@AJ-Acevedo
Copy link

Can someone point me to a working public version?

@RishikeshDarandale
Copy link
Author

RishikeshDarandale commented Apr 14, 2017

@philwareham , I have updated it to lunr-2.0.0. Please verify.
@AJ-Acevedo , here you go with working example of this search

@RishikeshDarandale RishikeshDarandale force-pushed the master branch 2 times, most recently from 16ba334 to 14cb97c Compare April 14, 2017 07:03
@philwareham
Copy link

@RishikeshDarandale this works great now, thanks very much!

@RishikeshDarandale
Copy link
Author

@philwareham, Glad to hear that!

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

Successfully merging this pull request may close these issues.

4 participants