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

fix: ドライブのデフォルト以外のソートでページネーションが正しくない ( #15270 ) #15278

Closed
wants to merge 2 commits into from

Conversation

tetsuya-ki
Copy link
Contributor

What

fix #15270

Why

ドライブでデフォルト以外のソートを選んだ時、ファイルが全部見れなくて困ってしまうため。

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/frontend Client side specific issue/PR packages/backend Server side specific issue/PR labels Jan 14, 2025
Copy link

codecov bot commented Jan 14, 2025

Codecov Report

Attention: Patch coverage is 0% with 37 lines in your changes missing coverage. Please review.

Project coverage is 39.31%. Comparing base (64501c6) to head (8e13aad).
Report is 19 commits behind head on develop.

Files with missing lines Patch % Lines
...es/backend/src/server/api/endpoints/drive/files.ts 0.00% 21 Missing ⚠️
packages/frontend/src/components/MkDrive.vue 0.00% 16 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #15278      +/-   ##
===========================================
- Coverage    41.12%   39.31%   -1.81%     
===========================================
  Files         1570     1566       -4     
  Lines       204607   198832    -5775     
  Branches      3693     3622      -71     
===========================================
- Hits         84151    78177    -5974     
- Misses      119851   120082     +231     
+ Partials       605      573      -32     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

このPRによるapi.jsonの差分
差分はありません。
Get diff files from Workflow Page

@tai-cha tai-cha changed the title fix #15270 fix: ドライブのデフォルト以外のソートでページネーションが正しくない ( #15270 ) Jan 14, 2025
@syuilo
Copy link
Member

syuilo commented Jan 14, 2025

クエリのパフォーマンス影響が気になりますね

@tetsuya-ki
Copy link
Contributor Author

まぁそうですよね。。。indexもあるidでページネーションしてたところをsizeやらnameやらのindexなしのやつにするなんて変更。。。
お試しするデータもないし、ダミーデータで増幅したとしても本番データと違うと指摘されたら反論できないので、一旦indexが効くであろうidによるソートだけに変更するプルリクエストを作成しようと思います。
・登録日(降順)
・登録日(昇順)
※ほかのソートは削除

Elasticsearchとかそういうやつに格納してソートやフィルタリングさせる方向でしか、パフォーマンス懸念を解消できる気がしないので、一旦これはクローズします。

お目汚しすみませんでした。

@tetsuya-ki tetsuya-ki closed this Jan 15, 2025
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
2 participants