search

smart-signature / MATATAKI_FE_Mobile

瞬MATATAKI frontend
https://www.matataki.io/
GNU Affero General Public License v3.0
27 stars 12 forks source link

【讚賞功能】去除幻影 #343

Closed yukiexe closed 5 years ago

yukiexe commented 5 years ago

开发需求清单 關於 "// 手动添加一个赞赏" 不要直接加一個沒有被後端確定已儲存的幻影，容易誤導開發人員或是使用者

解决方案 改成拿 sendComment 的回傳值去添加，請跟 @ganjingcun 協調回傳值設計

yukiexe commented 5 years ago

媽呀，誰寫的？

圖片

你看文檔，看有甚麼不一樣？ https://get-scatter.com/docs/api-request-arbitrary-signature

之所以沒出問題是因為這段根本沒用，後端自己寫了一套給前端.......

Linklinkguan commented 5 years ago

如果引入CPD重复代码检测工具来对完全一致copy过来的代码进行检查，会对这种状况有改善？

yukiexe commented 5 years ago

開頭的那項是個流程 bug 第二個是個未完成品 引入應該是沒法解決這問題

xiaotiandada commented 5 years ago

开发需求清单 關於 "// 手动添加一个赞赏" 不要直接加一個沒有被後端確定已儲存的幻影，容易誤導開發人員或是使用者

解决方案 改成拿 sendComment 的回傳值去添加，請跟 @ganjingcun 協調回傳值設計

已fix