peteranny commented 3 weeks ago

1385

這個 PR 是？

讓公司/職稱頁面的薪資標籤頁做分頁，包含搜尋

原本公司頁面的資料都放在 state.company.[name].status|data|error，資料包含薪資、面試、經驗。

由於薪資會獨立獲取，現將薪資移至 state.companyIndex.timeAndSalaryByName.[name].status|data|error。

Before	After

目前只有抽換 API，造成搜尋時會整頁重新載入。正常的使用體驗應該是只有下方表格重新載入。

由於這個 PR 已經太大，會另外開 PR 修正這個問題。這個 PR 只專注在抽換 pagination API。

peteranny commented 3 weeks ago

Just fixed some issues and should be okay now

barry800414 commented 3 weeks ago

另一個可以優化的地方是，只有第一次需要載入薪資相關的統計，切換不同頁數時不用重新載入。但同意這個 PR 已經不小，可以另外發 PR -> 另外開一張 enhancement 的票：https://github.com/goodjoblife/GoodJobShare/issues/1416

barry800414 commented 3 weeks ago

另外 debounce time 可能要調長一點

barry800414 commented 3 weeks ago

LGTM 沒想到需要的改動這麼大，感謝你

mark86092 commented 2 weeks ago

只有第一次需要載入薪資相關的統計

@barry800414 要不要開個 issue，超出分頁的 issue scope

barry800414 commented 2 weeks ago

只有第一次需要載入薪資相關的統計

@barry800414 要不要開個 issue，超出分頁的 issue scope

mark86092 commented 2 weeks ago

這個 PR 我也 cherry-pick 到 master branch，stage 站可以測試，看起來沒有跟 dev 相衝突