-
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
增加 route_boards 相關註解 #195
增加 route_boards 相關註解 #195
Conversation
Codecov Report
@@ Coverage Diff @@
## development #195 +/- ##
===============================================
+ Coverage 39.32% 43.00% +3.68%
===============================================
Files 22 25 +3
Lines 1030 1286 +256
===============================================
+ Hits 405 553 +148
- Misses 565 647 +82
- Partials 60 86 +26
Continue to review full report at Codecov.
|
LGTM |
// getBoardList get the board list for user with userID | ||
// Request URL: /v1/boards | ||
// API Doc: https://pttapp.cc/swagger/#/%E7%9C%8B%E6%9D%BF%E9%83%A8%E5%88%86/get_v1_boards |
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.
我覺得我們需要統一一個關於 handler 的註解格式,
例如我這裡用
// getBoardList ...
// Request URL: /v1/boards
// API Doc: ...
在 #142 用的是
// getUserFavorites is a http handler function which will get favorite list of user with userID
// to w. request path should be /v1/users/{{user_id}}/favorites
// Please see: ...
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.
我覺得please see 的用法比較好
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.
那 Request URL:
的部分呢?
另外,註解格式是否可以放在 documentation 內,方便大家參考?
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.
註解格式可以考慮放在 doc 裡,不過要放在哪裡又是一個要討論的點。
而且現在也沒有要求貢獻者寫註解,我個人是覺得還好。
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.
Ok!
Change title: API Doc to Please see.
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
👏 解決掉的 issue / Resolved Issues
📝 相關的 issue / Related Issues
⛏ 變更內容 / Details of Changes