bbs

Month: 2021-05

2021-05-01

pichuchen 21:49:27
Test sheet 的部分需要大家協助喔

2021-05-02

pichuchen 03:54:23
測試站目前的顯示和 ptt 有些不同,主要是目前不是用 pmore, 做個紀錄,避免我晚點睡起來忘掉

2021-05-05

pichuchen 09:23:58
這週影片會再晚一點上傳,我多一點時間想一下 C 轉成自然語言要怎麼表達比較好
陳昱廷 16:50:07
https://github.com/Ptt-official-app/Ptt-backend/pull/198

#198 增加 smtp provider 回傳

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/196|#196> *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/191|#191> *:pick: 變更內容 / Details of Changes* 增加 smtp provider

陳昱廷 16:50:28
這個希望大家參與討論一下
陳昱廷 16:51:07
需要集思廣益
pichuchen 18:20:36
我剛剛發現我前幾天打的review紀錄卡在Pending沒送出 Orz

2021-05-06

pichuchen 01:17:44
本週影片發出去了,然後又是發出去才想到有東西忘了講,pmore相關的忘了講
Rico 20:55:49
關於轉錄文章到看板一些問題

2021-05-07

Rico 20:54:08
feat: forward article usecase &amp; repository
想問大家這樣的方向對不對,或者哪裡還需要補的?
https://github.com/Ptt-official-app/Ptt-backend/pull/203

#203 feat: forward article usecase &amp; repository

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/154|#154> *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/66|#66> • <https://github.com/Ptt-official-app/Ptt-backend/issues/152|#152> • <https://github.com/Ptt-official-app/Ptt-backend/issues/153|#153> *:pick: 變更內容 / Details of Changes* • add forward article usecase • add forward article repository

2021-05-08

nickyanggg 17:25:13
再麻煩大家幫忙 review 一下,謝謝
https://github.com/Ptt-official-app/go-bbs/pull/84

#84 Feature/#83 實作發表文章

*:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/go-bbs/issues/83|#83> *:pick: 變更內容 / Details of Changes* • 在 bbs.go 中新增 WriteArticleConnector、NewArticleRecord、AddArticleRecordFileRecord • 新增 pttbbs/pttbbs_write_article_connector.go • 在 pttbbs/file.go 中新增 AppendFileHeaderFileRecord、NewFileRecord,其中比較不確定或是尚未實作的部分都有加上註解 • 在 pttbbs/file_test.go 中新增 TestAppendFileHeaderFileRecord、TestAppendFileHeaderFileRecordWithError

2021-05-09

titaneric 00:49:19
不好意思我想請問一下,我在改Ptt Backend時候遇到要對MockRepository與MockUsecase這兩個mock object兩邊都要新增類似MockUserDraft的struct這種情況,但我自己覺得只要在MockRepository新增就好,MockUsecase就呼叫MockRepository對應method就好(照我們的設計)。這邊我想討論MockUsecase的必要性,我看method測試function上,也沒有統一在MockRepository還是MockUsecase。如果說移除MockUsecase剩下MockRepository可能會比較好維護?
pichuchen 12:34:13
https://github.com/Ptt-official-app/Ptt-backend/pull/199

這個需要請大家協助確認

#199 實作發文 repo

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/180|#180> *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/180|#180> • <https://github.com/Ptt-official-app/Ptt-backend/issues/68|#68> *:pick: 變更內容 / Details of Changes* • usecase function `checkPermission` 新增傳入參數 context • 新增 checkPermission 關於發文的部份邏輯 • 新增 repo CreateArticle function

2021-05-10

titaneric 00:50:51
需要幫忙review以下PR,謝謝大家 https://github.com/Ptt-official-app/Ptt-backend/pull/204

#204 Feature/#167 實作草稿repo

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/167|#167> *:pick: 變更內容 / Details of Changes* • `GetUserDrafts`與`UpdateUserDraft`改傳回自定義的`UserDraft`,這需要在go-bbs內被實作 • 在`MockUsercase`與`MockRepository`新增`MockUserDraft` 以mock`UserDraft` *注意* • 目前`MockUserDraft` 是兩邊各定義相同的struct,之後可能要統一呼叫repo的

2021-05-11

amberfu 07:04:51
需要幫忙 review PR,感謝!
https://github.com/Ptt-official-app/go-bbs/pull/85|https://github.com/Ptt-official-app/go-bbs/pull/85

#85 Feature/#78 取得使用者設定

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/go-bbs/issues/78|#78> *:pick: 變更內容 / Details of Changes* • 新增 pttbbs/usersettings.go 與 pttbbs/usersettings_test.go 檔案 • 在 pttbbs/const.go 新增 uflags • 在 bbs.go 的 UserRecord interface 中新增 UserFlag() • 在 pttbbs/passwd.go 中修改 Userec struct 的 userFlag (原為 UserFlag) • 在 pttbbs/passwd_test.go 中修改所有 UserFlag 成為 userFlag • 在 serverlet/route_users_test.go 中新增 UserFlag() function

2021-05-12

Wombat007 17:11:52
ˋ7ㄚㄏˋ
Lex Liu 18:20:27
https://github.com/Ptt-official-app/go-bbs/issues/86

我想要嘗試這個 issue,但在 go-bbs 中沒看到 add comments 相關的實作,所以有幾個問題。
1. 是否需要在 ArticleRecord 中加入 comments 清單?
2. 是否需要考慮顏色的編碼? 參考 pptbbs/mbbsd/comments.c:4 FormatCommentString

#86 [主線] [PTT] 實作個人看板的歷史留言

*實作細節 / Details of Implement* • 基本上實作是參考 <https://github.com/Ptt-official-app/go-bbs/issues/64|#64> • 這個功能需要能夠掃瞄所有看板的所有文章,並列舉出所有屬於某個使用者的留言 • 因目前還未實作推文的功能,所以暫時的做法是透過 ReadBoardArticleFile 讀出文章內容後,判斷內容是否有包含使用者ID,有的話視為此使用者有留言。加上 TODO 之後再另起 issue 修改 • 新增 user_comment_record.go,定義 UserCommentRecord interface,至少實作以下 function • CommentOrder() int // 評論順序,從1開始,在 PTT 代表是幾樓的意思 • CommentTime() time.Time // 評論時間,發出推文的時間 • 在 bbs.go 新增 UserCommentConnector interface,實作以下 function • GetUserCommentRecordsPath(userID string) (string, error) // 取得儲放歷史留言快取資料庫的路徑 • ReadUserCommentRecordFile(name string) ([]UserCommentRecord, error) // 讀取快取資料,並輸出為[]UserCommentRecord • 在 bbs.go 新增 func (db *DB) GetUserCommentRecordFile(userID string) ([]UserCommentRecord, error) • 參考 <https://github.com/Ptt-official-app/go-bbs/blob/0a4883fa41988439603d95007d3ea25df682fddb/bbs.go#L399|GetUserArticleRecordFile>,先讀取快取資料庫的內容,若無資料則重新從所有看板的所有文章撈取。因目前沒有 comment_order 與 comment_time 的資料,可先回傳 empty value,再標記 TODO *期程 / Schedule* (希望這個任務大概什麼時候做完,如果會卡到其他地方的功能也請在這邊提出。) • 討論時間: 一週, 到 5/17 • 實作時間: 一週, 到 5/24 • 確認時間: 一週, 到 5/31 *相關文件 / Documents* • <https://github.com/Ptt-official-app/go-bbs/issues/64|#64> • <https://docs.google.com/document/d/18DsZOyrlr5BIl2kKxZH7P2QxFLG02xL2SO0PzVHVY3k/edit#heading=h.xtgbc6lc04uw|API文件>

2021-05-13

Julian Chu 19:12:39
需要幫忙 review PR 麻煩大家了
https://github.com/Ptt-official-app/Ptt-backend/pull/206

#206 Router benchmark

*:clap: 解決掉的 issue / Resolved Issues* *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/7|#7> • <https://github.com/Ptt-official-app/Ptt-backend/pull/124|#124> *:pick: 變更內容 / Details of Changes* • 補完 <https://github.com/Ptt-official-app/Ptt-backend/pull/124|#124> • 針對 ServeMux, gorillamux, httprouter 補上 benchmark

2021-05-14

Rico 19:20:17
稍微修正了一下,再麻煩大家指點迷津了

feat: forward article usecase &amp; repository 想問大家這樣的方向對不對,或者哪裡還需要補的? <https://github.com/Ptt-official-app/Ptt-backend/pull/203>