-
Notifications
You must be signed in to change notification settings - Fork 68
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
實作顯示看板資訊細部資訊 #138
實作顯示看板資訊細部資訊 #138
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
請 rebase |
Codecov Report
@@ Coverage Diff @@
## development #138 +/- ##
===============================================
- Coverage 38.88% 36.39% -2.49%
===============================================
Files 22 22
Lines 859 915 +56
===============================================
- Hits 334 333 -1
- Misses 475 530 +55
- Partials 50 52 +2
Continue to review full report at Codecov.
|
再麻煩幫忙review~ |
} | ||
|
||
func marshalBoardHeader(b bbs.BoardRecord, l *usecase.BoardPostLimitation) map[string]interface{} { | ||
func marshalBoardHeaderInfo(b bbs.BoardRecord, l *usecase.BoardPostLimitation) map[string]interface{} { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
可以改成marshalBoardHeaderWithInfo
或marshalBoardHeaderAndInfo
增進可讀性。marshalBoardHeaderInfo
乍看之下很像只有加入info本身
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已更改~
希望可以補上 getBoardSettings 測試 |
已補上~ |
👏 解決掉的 issue / Resolved Issues
📝 相關的 issue / Related Issues
⛏ 變更內容 / Details of Changes