feat: Query.search_companies add limits

mark86092 commented 3 weeks ago

Close #1319

這個 PR 是？

接上後端的 Query.search_companies 的 limit 參數

Screenshots

search bar

!有成功套用 limit!

{
  "query":"\n  query($companyName: String!, $hasData: Boolean!, $limit: Int) {\n    search_companies(query: $companyName, hasData: $hasData, limit: $limit) {\n      name\n      businessNumber\n      dataCount\n    }\n  }\n",
  "variables":{"companyName":"台","hasData":true,"limit":5}}

搜尋頁

!沒有改動，要不影響行為!

Query 保持一樣，不會卡 limit

{
  "query":"\n  query($companyName: String!, $hasData: Boolean!, $limit: Int) {\n    search_companies(query: $companyName, hasData: $hasData, limit: $limit) {\n      name\n      businessNumber\n      dataCount\n    }\n  }\n",
  "variables":{"companyName":"台積","hasData":true}
}

表單的 autocomplete

!有成功套用 limit!

{
  "query":"\n  query($companyName: String!, $hasData: Boolean!, $limit: Int) {\n    search_companies(query: $companyName, hasData: $hasData, limit: $limit) {\n      name\n      businessNumber\n      dataCount\n    }\n  }\n",
  "variables":{"companyName":"台大","hasData":false,"limit":10}
}

我應該如何手動測試？

[ ] 到上述的幾個網頁測試，開啟 console 看 network log

barry800414 commented 3 weeks ago

搜尋結果頁的部分，不加上 start & limit 嗎？還是這個部分會想要另外開 PR 做？

mark86092 commented 3 weeks ago

我本來是覺得沒必要改動，大家覺得呢

peteranny commented 3 weeks ago

搜尋結果可能要看分頁的問題，不知道會不會比較複雜，有需要的話可以另外弄沒關係

barry800414 commented 3 weeks ago

我在 dev 環境搜尋 "台灣"，mongodb 直接 OOM ，感覺有做分頁還是比較好一些不過我覺得沒有一定要是這個 PR

mark86092 commented 2 weeks ago

目前搜尋分頁的確會有問題，因為現在的搜尋是搜尋公司 + 搜尋職稱的結果 concat 在一起，以 dataCount 排序後再分頁

所以我覺得搜尋要分頁可能要有一個完整的 api，甚至要有計算結果放在 DB，才能減少後端一次把全部 query 出來的狀況

barry800414 commented 2 weeks ago

目前搜尋分頁的確會有問題，因為現在的搜尋是搜尋公司 + 搜尋職稱的結果 concat 在一起，以 dataCount 排序後再分頁

所以我覺得搜尋要分頁可能要有一個完整的 api，甚至要有計算結果放在 DB，才能減少後端一次把全部 query 出來的狀況

噢我理解你的意思，這個後端確實需要支援。我另外開票了 https://github.com/goodjoblife/WorkTimeSurvey-backend/issues/1059 https://github.com/goodjoblife/GoodJobShare/issues/1415

goodjoblife / GoodJobShare