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

[Task] New Search page #34

Merged
merged 2 commits into from
Aug 1, 2022
Merged

[Task] New Search page #34

merged 2 commits into from
Aug 1, 2022

Conversation

widyaput
Copy link
Contributor

@widyaput widyaput commented Aug 1, 2022

Story/Task

Create Searchpage

Summary

  • Added New search page
  • add fusejs in package

Test Plan

  • Input query in navbar, click search
  • Must find all posts related with title, author's name, and tag's name

Screenshot

Screenshot from 2022-08-01 21-16-20

Copy link
Contributor

@IloveNooodles IloveNooodles left a comment

Choose a reason for hiding this comment

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

approved with comments

keys: ['title', 'authors.name', 'tags.name'],
};

const fuse = new Fuse(data, options);
Copy link
Contributor

Choose a reason for hiding this comment

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

ini kalo datanya masi kosong gimanaa?

@@ -0,0 +1,16 @@
const Fuse = require('fuse.js');
Copy link
Contributor

Choose a reason for hiding this comment

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

ini hapus aja kalo ga dipake

@IloveNooodles IloveNooodles merged commit 1e0e03c into master Aug 1, 2022
@IloveNooodles IloveNooodles deleted the task/search branch August 1, 2022 15:40
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.

2 participants