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

Implemented the search using lunr.js - #14

Open
RishikeshDarandale wants to merge 2 commits into
mathaywarduk:masterfrom
RishikeshDarandale:master
Open

RishikeshDarandale wants to merge 2 commits into
mathaywarduk:masterfrom
RishikeshDarandale:master

Conversation

@RishikeshDarandale

Copy link
Copy Markdown

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
Copy Markdown

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
Copy Markdown
Author

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

@Vonfry

Vonfry commented Apr 10, 2017

Copy link
Copy Markdown

I have the same problem.

olivernn/lunr.js#254

@AJ-Acevedo

Copy link
Copy Markdown

Can someone point me to a working public version?

@RishikeshDarandale

RishikeshDarandale commented Apr 14, 2017

Copy link
Copy Markdown
Author

@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
Copy Markdown

@RishikeshDarandale this works great now, thanks very much!

@RishikeshDarandale

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