Ptt-official-app / Ptt-backend

PTT APP 的後端
BSD 3-Clause "New" or "Revised" License
208 stars 67 forks source link

修正沒有令牌的使用者取得資訊的錯誤回傳 #228

Closed ichiaohsu closed 3 years ago

ichiaohsu commented 3 years ago

👏 解決掉的 issue / Resolved Issues

⛏ 變更內容 / Details of Changes

修正回傳的錯誤訊息

codecov-commenter commented 3 years ago

Codecov Report

Merging #228 (0e6a4b6) into development (7e301b4) will decrease coverage by 0.23%. The diff coverage is 0.00%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #228      +/-   ##
===============================================
- Coverage        48.29%   48.05%   -0.24%     
===============================================
  Files               28       28              
  Lines             1404     1411       +7     
===============================================
  Hits               678      678              
- Misses             629      636       +7     
  Partials            97       97              
Impacted Files Coverage Δ
internal/delivery/http/route_users.go 47.79% <0.00%> (-1.39%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 7e301b4...0e6a4b6. Read the comment docs.

PichuChen commented 3 years ago

@ichiaohsu 可以幫我把你的執行結果截圖貼上來嗎?

y2468101216 commented 3 years ago

LGTM

ichiaohsu commented 3 years ago

@ichiaohsu 可以幫我把你的執行結果截圖貼上來嗎?

抱歉回覆晚了!執行結果是指 postman 的結果嗎?

y2468101216 commented 3 years ago

@ichiaohsu 可以幫我把你的執行結果截圖貼上來嗎?

抱歉回覆晚了!執行結果是指 postman 的結果嗎?

是的,可以參考 #233

ichiaohsu commented 3 years ago

@ichiaohsu 可以幫我把你的執行結果截圖貼上來嗎?

抱歉回覆晚了!執行結果是指 postman 的結果嗎?

是的,可以參考 #233

感謝!附圖如下: Screenshot 2021-06-24 at 23 59 47

@PichuChen 請參考

PichuChen commented 3 years ago

LGTM