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

feat: 検索でMeiliSearchを使えるようにする #10751

Closed
wants to merge 6 commits into from

Conversation

Steve-0628
Copy link

#10750 の出し直しです。

What

MeiliSearchが検索で使えるようにしました。新しいノートをMeiliSearchに追加するようにしました。Dockerも対応しました。

Why

検索がとても早くなるため。Related: #9323

Additional info (optional)

/packages/backend/src/server/api/endpoints/notes/search.tsの内容を参考に実装しましたが、従来の検索とは異なる結果が出る場合があります。これはMeiliSearchの仕様によるもので、設定を変更することによって改善できる場合があります。
テストは作っていません。
データベースにある既存のノートをMeiliSearchに入れることは可能ではありますが、このプルリクエストには含んでいません。

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR labels May 2, 2023
@codecov
Copy link

codecov bot commented May 2, 2023

Codecov Report

Merging #10751 (f1bbf61) into develop (a2e475f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head f1bbf61 differs from pull request most recent head b4b3373. Consider uploading reports for the commit b4b3373 to get more accurate results

@@           Coverage Diff            @@
##           develop   #10751   +/-   ##
========================================
  Coverage    78.35%   78.35%           
========================================
  Files          166      166           
  Lines        20680    20684    +4     
  Branches       373      373           
========================================
+ Hits         16203    16207    +4     
  Misses        4477     4477           
Impacted Files Coverage Δ
packages/frontend/src/components/MkPagination.vue 100.00% <100.00%> (ø)

@Steve-0628
Copy link
Author

#10755 でしゅいろさんが実装したようなので閉じます

@Steve-0628 Steve-0628 closed this May 3, 2023
@syuilo
Copy link
Member

syuilo commented May 3, 2023

#10755 がマージされる保証がない

@Steve-0628
Copy link
Author

#10755 がマージされなかったら開け直すことにします

@Steve-0628 Steve-0628 reopened this May 3, 2023
@github-actions github-actions bot requested a review from rinsuki May 3, 2023 09:38
@Steve-0628
Copy link
Author

検索箱に、たとえばfrom:@user@domain.tldみたいな入力をすると、filterに良い感じにして検索してくれるやつを実装したいなって思っています

@syuilo
Copy link
Member

syuilo commented May 4, 2023

#10755 をマージしたためこちらはcloseします 🙏🏻

@syuilo syuilo closed this May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants