search

Ptt-official-app / go-bbs

BBS file database manager system library in Golang.
Apache License 2.0
80 stars 43 forks source link

Implement #95 fix UserCommentRecord. #96

Closed nickyanggg closed 3 years ago

nickyanggg commented 3 years ago

👏 解決掉的 issue / Resolved Issues

⛏ 變更內容 / Details of Changes

PichuChen commented 3 years ago

UserComment 這個 interface 這樣開感覺有問題，他應該不必要實作 ArticleRecord 這個 method? 另外是 method name 的部分

CommentOwner => Owner CommentIP => IP

這樣會不會比較好？

nickyanggg commented 3 years ago

UserComment 這個 interface 這樣開感覺有問題，他應該不必要實作 ArticleRecord 這個 method? 另外是 method name 的部分

CommentOwner => Owner CommentIP => IP

這樣會不會比較好？

了解，因為這次 pr 主要是想說可能需要實作 ArticleRecord，所以我還是先關 pr，取名的部分可能之後再修改。

PichuChen commented 3 years ago

LGTM