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

改寫部分最萌亂鬥大賽相關程式碼 & 重構路由相關程式碼 #577

Merged
merged 3 commits into from
Jan 11, 2019

Conversation

frozenmouse
Copy link
Contributor

亂鬥部分(合併 #576 ):

  1. 將 panelFolder 的使用 key 前綴以目前 route 名稱以減低撞名的機會
  2. 亂鬥大賽資訊頁面 arenaInfo 將參賽者列表以分頁呈現,並重寫資料撈取方式,
    避免報名人數過多時造成客戶端與 server 的效能問題
  3. 為加入撈取資料時 server 端的排序支援,廢除 dbArenawinnerList
    將名次資訊移至各個參賽者的 rank 欄位
  4. 同樣為完成排序支援,各參賽者的資料加入 totalInvestedAmount 欄位記錄總投資額,
    以此取代先前在 client 端的即時運算
  5. 加入 server 端 method 提供戰鬥紀錄篩選功能的支援
  6. 修正新的亂鬥大賽資料在產生時,其結束時間會比原本預計少一週的問題
  7. client/arenaInfo/arenaInfo.html 各 template 整理拆分為對應檔案
  8. 參賽者列表加入響應式表格支援
  9. 亂鬥結束之後不再移除參賽者的投資者列表

路由部分:

  1. 由於目前只有 client 端能夠使用此路由,故移除所有原本以為會在 server 端執行的 code,避免誤解
  2. DocHead 設定頁面標題的動作移至 layout 進行
  3. 移除不必要的 route group name
  4. 將公司列表、新創列表與帳號資訊原本的轉址動作,移至轉址後目標頁面的樣版處理,
    避免因資料不齊全而造成轉址動作無法處理或頁面顯示失敗
  5. 額外實作季度報告與亂鬥大賽頁面未指定 id 時的轉址動作
  6. 將規則討論議程列表之路由從 /ruleDiscuss/list 簡化為 /ruleDiscuss/,以移除多餘的轉址動作

因目前實作是全體共用一個 ReactiveDict,
如此至少可避免不同 route 下有同樣 name 的 panelFolder 撞 key,造成操作上的誤會
1. 亂鬥大賽資訊頁面 `arenaInfo` 將參賽者列表以分頁呈現,並重寫資料撈取方式,
   避免報名人數過多時造成客戶端與 server 的效能問題
2. 為加入撈取資料時 server 端的排序支援,廢除 `dbArena` 的 `winnerList`,
   將名次資訊移至各個參賽者的 `rank` 欄位
3. 同樣為完成排序支援,各參賽者的資料加入 `totalInvestedAmount` 欄位記錄總投資額,
   以此取代先前在 client 端的即時運算
4. 加入 server 端 method 提供戰鬥紀錄篩選功能的支援
5. 修正新的亂鬥大賽資料在產生時,其結束時間會比原本預計少一週的問題
6. 將 `client/arenaInfo/arenaInfo.html` 各 template 整理拆分為對應檔案
7. 參賽者列表加入響應式表格支援
8. 亂鬥結束之後不再移除參賽者的投資者列表
1. 由於目前只有 client 端能夠使用此路由,故移除所有原本以為會在 server 端執行的 code,避免誤解
1. 將 `DocHead` 設定頁面標題的動作移至 layout 進行
2. 移除不必要的 route group name
3. 將公司列表、新創列表與帳號資訊原本的轉址動作,移至轉址後目標頁面的樣版處理,
   避免因資料不齊全而造成轉址動作無法處理或頁面顯示失敗
4. 額外實作季度報告與亂鬥大賽頁面未指定 id 時的轉址動作
5. 將規則討論議程列表之路由從 `/ruleDiscuss/list` 簡化為 `/ruleDiscuss/`,以移除多餘的轉址動作
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