bbs

Month: 2021-04

2021-04-02

gogog22510 13:01:47
麻煩大家幫忙review,感謝!
https://github.com/Ptt-official-app/Ptt-backend/pull/159

#159 Implement http for user information and enhance test case

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/100|#100> *:memo: 相關的 issue / Related Issues* • #主 issue <https://github.com/Ptt-official-app/Ptt-backend/issues/52|#52> *:pick: 變更內容 / Details of Changes* • 修改error message • 檢查token permission for real name and money • 補齊testcode

testcase 的部分應該拆開來比較好
再來是 readRealname 的permission 設計,應該是在 ISSUE 上面討論再來這樣設計
在需求上好像沒有這樣設計
gogog22510 2021-04-02 23:51:55
Permission是因為我看到api doc上面有標注,不需要的話我可以拿掉
gogog22510 2021-04-02 23:52:53
test case 拆開的部分可以舉個例子嗎?我原本寫的可能不太好,應該可以直接比對整個data map
testcase 應該可以參考這樣設計? https://github.com/Ptt-official-app/Ptt-backend/pull/146/files
> Permission是因為我看到api doc上面有標注,不需要的話我可以拿掉
是文件上的哪段?
> Permission是因為我看到api doc上面有標注,不需要的話我可以拿掉
是文件上的哪段?
gogog22510 2021-04-03 23:54:12
在/v1/users/{{user_id}}/information裡有提到需要檢查permission
gogog22510 2021-04-03 23:58:12
參考link裡的test case只有比對map裡的一筆資料,我想要比對所有的field,我可以改成直接比對map本身
❤️ 1

2021-04-03

2021-04-09

titaneric 14:50:23
需要跟大家討論一下API規格 https://docs.google.com/document/d/18DsZOyrlr5BIl2kKxZH7P2QxFLG02xL2SO0PzVHVY3k/edit?disco=AAAAINjPMDY 這會影響到https://github.com/Ptt-official-app/Ptt-backend/issues/155 以及 https://github.com/Ptt-official-app/Ptt-backend/issues/152 實作

#155 [主線] [PTT] 實作轉錄信箱 http module

#152 [主線] [PTT] 實作轉錄看板文章 -- delivery/http

是參數的那個問題嗎?
我現在正在嘗試轉錄看本文章的 repository,我要怎麼測我寫的是可以正常運作的?
另外有個問題,回應文章跟轉錄看板文章似乎有點像。
我實際去看 PTT,
回應文章 (y) 的確有分到看板跟信箱;
轉錄 (^X) 似乎只有轉到別的看板而已。

所以實作上,回應文章到看板跟轉錄是一樣的東西囉?
可以hint一下原ptt的檔案位置嗎 方便看
> 我現在正在嘗試轉錄看本文章的 repository,我要怎麼測我寫的是可以正常運作的?
我覺得可以看一下登入之類的,基本上就是單元測試,所以對應的函式有被呼叫到就算過關?
> 所以實作上,回應文章到看板跟轉錄是一樣的東西囉?
應該是,但是還是有細節差異,像是他們下的標題以及附加的文字,轉錄對原文章也會產生修改,回應不會。
👀 1

2021-04-10

titaneric 17:10:22
需要幫忙review以下PR,謝謝 https://github.com/Ptt-official-app/Ptt-backend/pull/169

#169 Feature/#152 forward article

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/152|#152> *:pick: 變更內容 / Details of Changes* *Route* • 在delivery/http/route.go中的`postBoards` method,針對action = "forward_article" 呼叫delivery.ForwardArticle • 在delivery/http裡新增`route_forward_article.go`實作`ForwardArticle` method,呼叫usecase.ForwardArticle • 在usecase/token.go 新增`PermissionForwardArticle` 權限 • 在uscase/usecase.go新增`ForwardArticle` method至Usecase interface • 在usecase/article.go裡面新增`ForwardArticle` method *Test* • 在delivery/http/route_article_test.go裡面新增`forwardArticle` mock test method • 在delivery/http裡新增`route_forward_article_test.go` • 更新swagger.yml action 內容

nickyanggg 19:27:06
再麻煩幫忙 review,謝謝
https://github.com/Ptt-official-app/Ptt-backend/pull/170

#170 Feature/#166 取得儲存草稿 - delivery/http

*:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/166|#166> *:pick: 變更內容 / Details of Changes* • 修改 route.go 中的 parseUserPath • 修改 route_users_test.go 中的 TestParseUserPath • 在 route_users.go 中新增 getUserDrafts • 在 route_users_test.go 中新增 TestGetUserDrafts

2021-04-13

陳昱廷 12:22:36
我增加了一些增加註解的 issue 再 ptt-backend
陳昱廷 12:22:56
有興趣者可以再 issue 下面回應
陳昱廷 12:23:02
我會再指派給你
👍 2

2021-04-14

pichuchen 03:59:52
有需要人力的話可以從人力狀態表上面找,發現還滿有效率的
pichuchen 04:52:01
本週影片寄出去了
陳昱廷 14:48:04
<!channel> 目前準備製作貢獻者名單,大家快一起來留下你的名字 https://github.com/Ptt-official-app/Ptt-backend/issues/182

#182 [一般] 貢獻者名單

*實作細節 / Details of Implement* 因為第一期 milestone 即將結束,目前準備製作一份貢獻者名單加到專案裡面。 包括但不限於 coding,撰寫文件等 歡迎大家提名自己到名單內 *期程 / Schedule* 無,但會定時 merge 到 development *相關文件 / Documents* CONTRIBUTORS.md 提名格式如下: 1. 名字 (網路名稱):必填 2. GitHub 帳號:非必填 3. 社交帳號:非必填,請給連結 4. 貢獻內容:必填

👍 4 ❤️ 2
陳昱廷 14:59:35
go-bbs 的在這裡
陳昱廷 14:59:36
https://github.com/Ptt-official-app/go-bbs/issues/76

#76 [一般] 貢獻者名單

*實作細節 / Details of Implement* 目前準備製作一份貢獻者名單加到專案裡面。 包括但不限於 coding,撰寫文件等 歡迎大家提名自己到名單內 *期程 / Schedule* 無,但會定時 merge 到 development *相關文件 / Documents* CONTRIBUTORS.md 1. 名字 (網路名稱):必填 2. GitHub 帳號:非必填 3. 社交帳號:非必填,請給連結 4. 貢獻內容:必填

pichuchen 18:37:48
https://github.com/Ptt-official-app/Ptt-backend/pull/169

這個目前還缺一個Reviewer

#169 Feature/#152 forward article

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/152|#152> *:pick: 變更內容 / Details of Changes* *Route* • 在delivery/http/route.go中的`postBoards` method,針對action = "forward_article" 呼叫delivery.ForwardArticle • 在delivery/http裡新增`route_forward_article.go`實作`ForwardArticle` method,呼叫usecase.ForwardArticle • 在usecase/token.go 新增`PermissionForwardArticle` 權限 • 在uscase/usecase.go新增`ForwardArticle` method至Usecase interface • 在usecase/article.go裡面新增`ForwardArticle` method *Test* • 在delivery/http/route_article_test.go裡面新增`forwardArticle` mock test method • 在delivery/http裡新增`route_forward_article_test.go` • 更新swagger.yml action 內容

2021-04-15

陳昱廷 09:35:54
我 review 完了
陳昱廷 09:36:03
#169

2021-04-17

gogog22510 03:32:39
可以麻煩幫忙review,感謝
https://github.com/Ptt-official-app/Ptt-backend/pull/159

#159 Implement http for user information and enhance test case

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/100|#100> *:memo: 相關的 issue / Related Issues* • #主 issue <https://github.com/Ptt-official-app/Ptt-backend/issues/52|#52> *:pick: 變更內容 / Details of Changes* • 修改error message • 補齊testcode

🙌 2
nickyanggg 15:08:02
再麻煩大家幫忙 review,感謝
https://github.com/Ptt-official-app/Ptt-backend/pull/185

#185 Feature/#166 儲存、刪除草稿 - delivery/http

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/166|#166> *:pick: 變更內容 / Details of Changes* • 在 route_users.go 中新增 postUsers 、postUserDrafts、updateUserDraft、deleteUserDraft • 在 route_users_test.go 中新增 TestUpdateUserDraft、TestDeleteUserDraft • 在 usecase/token.go 中新增 PermissionUpdateDraft、PermissionDeleteDraft

🙌 1
pichuchen 22:06:56
我把Ptt-backend 在 pttapp.cc 上面的版本做更新了
pichuchen 22:07:05
然後有撿到一個錯誤訊息
```
2021/04/14 01:29:02 path: /home/bbs/home/T/TRUE0422/.fav [0/2051]
2021/04/14 01:29:02 http: panic serving [::1]:36532: runtime error: invalid memory address or nil pointer dereference
goroutine 5785 [running]:
net/http.(*conn).serve.func1(0xc000099220)
/usr/local/go/src/net/http/server.go:1801 +0x147
panic(0x6ee5a0, 0x9753f0)
/usr/local/go/src/runtime/panic.go:975 +0x47a
github.com/Ptt-official-app/go-bbs/pttbbs.(*Connector).ReadUserFavoriteRecordsFile(0xc000013070, 0xc000416a80, 0x1e, 0xc000416a80, 0x1e, 0x0, 0x0, 0xc0000aab40)
/root/go/pkg/mod/github.com/!ptt-official-app/go-bbs@v0.2.6/pttbbs/pttbbs.go:63 +0x100
github.com/Ptt-official-app/go-bbs.(*DB).ReadUserFavoriteRecords(0xc000013b30, 0xc00007ab1e, 0x8, 0x8, 0x10, 0x7f1eff5dc108, 0xc0001023b0, 0xc0000f59e0)
/root/go/pkg/mod/github.com/!ptt-official-app/go-bbs@v0.2.6/bbs.go:192 +0x1c8
github.com/Ptt-official-app/Ptt-backend/internal/repository.(*repository).GetUserFavoriteRecords(0xc00004c840, 0x7a2420, 0xc0000160b8, 0xc00007ab1e, 0x8, 0x40f9b8, 0x10, 0x6df0e0, 0x728801, 0xc0001023b0)
/root/Ptt-backend/internal/repository/user.go:55 +0x45
github.com/Ptt-official-app/Ptt-backend/internal/usecase.(*usecase).GetUserFavorites(0xc00006b2c0, 0x7a2420, 0xc0000160b8, 0xc00007ab1e, 0x8, 0x1, 0xc000296810, 0x0, 0x0, 0x5)
/root/Ptt-backend/internal/usecase/user.go:26 +0x63
github.com/Ptt-official-app/Ptt-backend/internal/delivery/http.(*httpDelivery).getUserFavorites(0xc000104080, 0x7a1ce0, 0xc0000e8620, 0xc00008cf00, 0xc00007ab1e, 0x8)
/root/Ptt-backend/internal/delivery/http/route_users.go:74 +0x196
github.com/Ptt-official-app/Ptt-backend/internal/delivery/http.(*httpDelivery).getUsers(0xc000104080, 0x7a1ce0, 0xc0000e8620, 0xc00008cf00)
/root/Ptt-backend/internal/delivery/http/route_users.go:17 +0x247
github.com/Ptt-official-app/Ptt-backend/internal/delivery/http.(*httpDelivery).routeUsers(...)
/root/Ptt-backend/internal/delivery/http/route.go:72
net/http.HandlerFunc.ServeHTTP(0xc0001021e0, 0x7a1ce0, 0xc0000e8620, 0xc00008cf00)
/usr/local/go/src/net/http/server.go:2042 +0x44
net/http.(*ServeMux).ServeHTTP(0xc00004c880, 0x7a1ce0, 0xc0000e8620, 0xc00008cf00)
/usr/local/go/src/net/http/server.go:2417 +0x1ad
net/http.serverHandler.ServeHTTP(0xc0000e80e0, 0x7a1ce0, 0xc0000e8620, 0xc00008cf00)
/usr/local/go/src/net/http/server.go:2843 +0xa3
net/http.(*conn).serve(0xc000099220, 0x7a23e0, 0xc00004cd80)
/usr/local/go/src/net/http/server.go:1925 +0x8ad
created by net/http.(*Server).Serve
/usr/local/go/src/net/http/server.go:2969 +0x36c```
這個解決了嗎,我那天看感覺是沒有問題的
還沒再次觸發

2021-04-18

pichuchen 13:55:18
https://github.com/Ptt-official-app/go-bbs/issues/78

使用者設定的部分在go-bbs上可以開始討論了

#78 [主線] [PTT] 取得使用者設定

*實作細節 / Details of Implement* • 請見 <https://github.com/ptt/pttbbs/blob/5715b35f510f48eb5092d32882f1aa09181dc3a1/include/pttstruct.h#L57|userec_t> 中的 uflag 和 userlevel *期程 / Schedule* (希望這個任務大概什麼時候做完,如果會卡到其他地方的功能也請在這邊提出。) • 討論時間: 一天 4/19 • 實作時間: 一週 4/25 • 確認時間: 一週 5/2 *相關文件 / Documents* <https://github.com/ptt/pttbbs/blob/5715b35f510f48eb5092d32882f1aa09181dc3a1/include/pttstruct.h#L57|userec_t>

👍 1
pichuchen 15:09:49
https://github.com/Ptt-official-app/go-bbs/issues/79

發文限制和細部資訊的 go-bbs ISSUE 有開出來了

#79 [主線] [PTT] 實作看板發文限制以及細部資訊

*實作細節 / Details of Implement* • <https://github.com/ptt/pttbbs/blob/5715b35f510f48eb5092d32882f1aa09181dc3a1/mbbsd/board.c#L338|board.c> 檔案的 b_config 上面有看板限制和細部資訊,兩邊基本上是一起被取出的,所以可以放在同個 Method 出來。 <https://user-images.githubusercontent.com/600238/115137279-df118b80-a057-11eb-8ded-e1d964caf57d.png|截圖 2021-04-18 下午3 07 59> *期程 / Schedule* (希望這個任務大概什麼時候做完,如果會卡到其他地方的功能也請在這邊提出。) • 討論時間: 一天 4/19 • 實作時間: 一週 4/26 • 確認時間: 一週 5/3 *相關文件 / Documents*

os369510 21:56:41
@os369510 has joined the channel

2021-04-20

whitefloor 19:27:08
https://github.com/Ptt-official-app/Ptt-backend/pull/187

#164 需要幫忙review 感謝

#187 implement GetPopularBoards usecase and add mock test in board_mock_test

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/164|#164> • close <https://github.com/Ptt-official-app/Ptt-backend/issues/181|#181> *:memo: 相關的 issue / Related Issues* • 主issue <https://github.com/Ptt-official-app/Ptt-backend/issues/51|#51> *:pick: 變更內容 / Details of Changes* • usecase 1. 新增usecase method:GetPopularBoards 2. 新增usecase test:TestGetPopularBoards • <https://github.com/Ptt-official-app/Ptt-backend/issues/181|#181> : 由於直接使用GetBoards取得資料再由usecase進行篩選,所以沒有新增function

👀 1

2021-04-21

pichuchen 03:04:16
這週的影片錄好了,不過我忘記把行為測試的內容加上去了......
pichuchen 03:25:34
本週影片寄出了,記得要填問卷,然後Readme上面的 QA 募集中
陳昱廷 16:44:34
這個解決了嗎,我那天看感覺是沒有問題的

2021-04-22

golopot 00:28:18
@golopot has joined the channel
pichuchen 14:47:03
https://www.ithome.com.tw/news/143963?fbclid=IwAR1IGjYn53Op3uSGyWYGgx2-tziSCEAI8fujW-UU25qamMWzpvh4yBUIXUs

我們好像有用到的樣子?雖然開源不怕偷啦

iThome

Codecov被駭,殃及數百家企業客戶及IBM等科技公司

目前美國政府調查發現,數百家Codecov客戶可能已被竊走內部軟體開發專案資料,資料外洩規模超出Codecov原先預期的範圍

andylee1013 2021-04-22 19:59:41
有喔 有些PR有看到report
不過又看到熟悉的關鍵字Solarwinds 😂
😮 1

2021-04-23

wagaru 09:36:31
再請幫忙 review 了 https://github.com/Ptt-official-app/Ptt-backend/pull/189

#189 add forwardArticleToEmail

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/156|#156> *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/156|#156> • <https://github.com/Ptt-official-app/Ptt-backend/issues/67|#67> *:pick: 變更內容 / Details of Changes* • 在 config_default.toml 新增 mailDriver 設定,預設為 smtp • 在 usecase.NewUsecase 會依據 config.MailDriver 設定 mail • 在 usecase 新增 updateMail 替換不同的 mail • 新增 mail.go 定義 Mail interface

Chang Chen Chien 10:09:27
請問 y94
Chang Chen Chien 10:11:48
大家好,我想幫忙 [主線] [PTT] 增加 route_borad 相關註解 這個 issue
請問在 `https://pttapp.cc/v1/boards` 這個 api 為什麼我會得到 `token_invalid` 錯誤呢? 從 `getBoardList()` 這個 handler 來看,如果我是未登入的使用者的話 應該是以 `guest` 身份進行的不是嗎?

#175 [主線] [PTT] 增加 route_borad 相關註解

*實作細節 / Details of Implement* 主要是增加 route_borad 相關註解 可參考 <https://github.com/Ptt-official-app/Ptt-backend/issues/110|#110> *期程 / Schedule* • 討論時間: 三天 • 實作時間:一天 • 確認時間: 一天 *相關文件 / Documents* route_boards.go, route_boards_test.go , route_boards_mock_test.go

guest 的話應該還是要以 guest 然後無密碼去取得 token?
Chang Chen Chien 2021-04-24 16:30:35
恩對,因為在我的理解裡,guest 也是要能拿到 board list 的,而不是 得到 `token_invalid` 就什麼都看不到>
Chang Chen Chien 2021-04-24 16:30:45
不知道這樣理解是否正確?
不太正確,因為不是所有 BBS 站 guest 都能拿到 board list, 是最近幾年 ptt 出現網頁版,所以才會拿到。

另外不同權限的人拿到的 board list 都會有點不同,會有些隱藏看板看不到。
Chang Chen Chien 2021-04-26 08:31:01
了解了, 反正我先送 PR 給大家 review ,在 github g上再討論哪邊需要修正
Chang Chen Chien 10:16:26
雖然裡面有個對 `if !supportGuest()` 的 check, 但是 `supportGuest()` 這個 function 沒有實作, 所以不太懂這裡的邏輯。
哪邊?
Chang Chen Chien 2021-04-24 16:26:36
抱歉,是我沒注意到。
`supportGuest()` 的實作被放在 `internal/delivery/http/route.go` 裡面
那個應該是確認這個BBS站之不支援 Guset 登入?
Chang Chen Chien 2021-04-26 08:29:59

2021-04-24

nickyanggg 09:59:33
再麻煩大家幫忙 review,感謝!
https://github.com/Ptt-official-app/Ptt-backend/pull/192

#192 Feature/#168 儲存草稿 - usecase

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/168|#168> *:pick: 變更內容 / Details of Changes* • 在 usecase/user.go 中新增 GetUserDrafts 、UpdateUserDraft、DeleteUserDraft、isValidDraftID(目前合法的 draftID 為 0~9) • 在 usecase/user_test.go 中新增 TestGetUserDrafts、TestUpdateUserDrafts

2021-04-25

whitefloor 13:02:09
請問github上面除了按re-request review之外有其他的方式可以通知別人review嗎@@?
andylee1013 2021-04-25 16:38:07
tag人算嗎 用@
andylee1013 16:50:38
麻煩大家幫忙看看這個PR 是跟 Issue #71 有關 我覺得可以討論一下這個做法是否適合 感謝~~
https://github.com/Ptt-official-app/Ptt-backend/pull/193

#193 新增 Dummy Logger for Testing

*:clap: 解決掉的 issue / Resolved Issues* *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/71|#71> *:pick: 變更內容 / Details of Changes* • `logging` • 新增 `DummyLogger` 結構,method 皆為空函式 • `main` • 修改 `NewRepository` `NewUsecase` `NewHTTPDelivery` 傳入變數 `logger` 作為相依 _(Dependency)_ • `http` • 修改測試函式皆使用 `DummyLogger` 為相依 • `repository` • 修改測試函式皆使用 `DummyLogger` 為相依 • `usecase` • 修改測試函式皆使用 `DummyLogger` 為相依 _注意:此PR目的是演示一下 mock logger 的概念_

andylee1013 2021-04-25 16:52:43
自己回復一下~關於mock logger 的做法有很多種,另一個做法是用 mockgen 產生一個實作,但是在Testing code裡面就會每個test case都要去Expect 所有的logger method。
我是覺得他可以丟到 global 去這樣
andylee1013 2021-04-25 17:01:25
所謂的global是指什麼
andylee1013 2021-04-25 17:03:10
我有想過 mock_logger 裡面放一個 var xx = &DummyLogger{} 就給所有test case用
> 我有想過 mock_logger 裡面放一個 var xx Logger = &DummyLogger{} 就給所有test case用
類似這樣,可能就是有個 defaultLogger 像是 http.defaultClient 這樣
andylee1013 2021-04-25 18:24:36
恩恩 其實實際的logger 跟DummyLogger 都可做這件事
andylee1013 2021-04-25 18:25:02
我覺得只要大家有個共識就可以
andylee1013 18:06:35
請大家幫忙reivew~~ 感謝~~
https://github.com/Ptt-official-app/Ptt-backend/pull/190

#190 Implement get user info usecase

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/165|#165> *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/52|#52> *:pick: 變更內容 / Details of Changes* • repository • 新增 `GetUserInformation()` • 注意 function 尚未實作 • usecase • 修改 `GetUserInformation()` 回傳 repository 的 return 值 • 新增 `TestGetUserInformation()`

👍 1
chhsiao1981 23:18:49
hi all~

在這裡打個廣告~

在 go-openbbsmiddleware / go-pttbbs 這一線裡

最近根據 https://ocf.tw/p/pttid 裡的資料加上了測試資料~
總共有 170000+ 個 user (test1~test170000. 密碼都是 123123)
8k+ 個板.
其中 Gossiping / Baseball / C_Chat / HatePolitics / Lifeismoney / LoL / MobileComm / NBA / Stock / sex 板都有資料.
(Gossiping: 100k+ 文章)

很歡迎大家從以下 link 測試:

https://www.devptt.site
https://www.devptt.site/boards
https://www.devptt.site/board/17_Gossiping/articles

測試資料 (/home/bbs) 在以下 link:
https://drive.google.com/file/d/1lHuqOYpPDmKayYAaf3UIiLRV1wCjF6bc/view

也很歡迎大家一起幫忙

https://github.com/Ptt-official-app/go-openbbsmiddleware
https://github.com/Ptt-official-app/go-pttbbs

這兩個 projects~

ocf.tw

PTT ID Profiling | 財團法人開放文化基金會 (OCF)

網路上的不實資訊、網軍操作帶風向的狀況層出不窮,批踢踢實業坊 (PTT) 的政治類看板, 比起其他看板,帶風向的情形更加嚴重,網路上已經有很多專家利用爬蟲工具,自行搜尋分析出活動異常的 PPT 使用者帳號。本計畫希望能網羅各方資源之外,也能開發出套完整的開源工具,讓使用者自行析判斷誰是誰非。

devptt.site

React App

Web site created using create-react-app

devptt.site

React App

Web site created using create-react-app

devptt.site

React App

Web site created using create-react-app

2021-04-26

pichuchen 01:18:18
https://yushuanhsieh.github.io/post/2021-04-24-go-calling-convention

不說我還以為 golang 的參數一直都是放在register的...

yushuanhsieh.github.io

Switch to a register-based calling convention for Go functions

自己對於 memory layout 相關議題都蠻感興趣的,而這次 Go 1.17 有一項效能改善的 proposal: switch to a register-based calling convention for Go functions 剛好跟 memory 有相關,因此就看了一下 proposal 文件介紹,不但複習了在計算機架構中曾接觸到的 calling convention 知識,也對於 Go 內部機制有更多認識。

saker 02:32:47
@saker.lin has joined the channel
🙌 1

2021-04-27

Chang Chen Chien 10:28:12
請大家幫忙 review 謝謝
是關於註解的 PR
https://github.com/Ptt-official-app/Ptt-backend/pull/195

#195 增加 route_boards 相關註解

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/175|#175> *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/109|#109> • <https://github.com/Ptt-official-app/Ptt-backend/issues/110|#110> *:pick: 變更內容 / Details of Changes* • 新增註解 • 新增 handler 的 Request URL 與 API Doc link • 對還沒列入 swagger 的 API Doc link 加上 TODO prefix

pichuchen 17:13:52
https://github.com/Ptt-official-app/Ptt-backend/issues/154

這個 ISSUE 的實際作法需要討論,這是轉錄看板文章的 ISSUE, 現有轉錄行為除了會在新版新增一篇文章以外,還會對原文章新增一行修改,因此我認為他比較多的動作其實是在 usecase上面要做動作

#154 [主線] [PTT] 實作轉錄看板文章 -- repository

*實作細節 / Details of Implement* • 此為 <https://github.com/Ptt-official-app/Ptt-backend/issues/66|#66> 子任務 • 實作 <https://github.com/Ptt-official-app/Ptt-backend/blob/development/internal/repository/repository.go|repository.go> • 因為 <https://github.com/Ptt-official-app/go-bbs|go-bbs> 尚未完成,需要先 mock 一個暫時性的結果等待 go-bbs 完成之後進行串接。 *期程 / Schedule* • 討論時間: 一週, 到 4/7 • 實作時間:一週, 到 4/14 • 確認時間:一週, 到 4/21 *相關文件 / Documents* • 主issue: <https://github.com/Ptt-official-app/Ptt-backend/issues/66|#66> • <https://docs.google.com/document/d/18DsZOyrlr5BIl2kKxZH7P2QxFLG02xL2SO0PzVHVY3k/edit|PTT後端系統協定>

pichuchen 17:36:54
https://github.com/DeanXu2357/Ptt-backend/blob/c039d29ca07d95b72269310fa6f414b0ab6bdc93/internal/usecase/article.go#L21

目前在推文的程式碼中好像沒有檢查對應的權限,這部分有人要開ISSUE嗎?

internal/usecase/article.go:21

``` //result, err := usecase.repo.AppendComment(ctx, userID, boardID, filename, appendType, text) ```

pichuchen 22:39:11
https://docs.google.com/spreadsheets/d/1ITDYuj6TjeodZDtnXPrctZ6We6Mz_tXosmR_O1TCkXo/edit#gid=0

我多開了一個函式註解的 Google Sheet, 因為接下來會需要去讀 c 的程式碼,因此可以減少重覆解讀同一段程式碼的人力負擔
pichuchen 23:26:18
https://docs.google.com/spreadsheets/d/1uo4AJuSi5xTXEht2o2EHogLivCJlJqlLaeqoj1RceDY/edit#gid=0

另外這是行為測試用的 Sheet 在接下來 MS1 結束之後拿來增強信心用的

2021-04-28

pichuchen 01:38:37
本週影片寄出了,再請大家注意信箱這樣
Chang Chen Chien 09:23:00
@pichuchen pttapp.cc 的憑證過期囉
感謝提醒,開回來了
陳昱廷 14:40:22
https://github.com/Ptt-official-app/Ptt-backend/pull/200

#200 根據 whitefloor 調整了既有的說明文檔

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/151|#151> *:pick: 變更內容 / Details of Changes* 根據 whitefloor 調整了一些文檔上的問題 主要的更動如下: 1. 將歷史緣由跟開發原因搬進 FOR_DEVELOPERS.md 裡面 我覺得作為一個專案的門面,應當盡量簡單扼要說明這個專案的的功能。 2. 直接增加了如何參與這個專案 由於希望此專案不要變成一人開發,需要用更簡單的方法吸引人進來參與 3. 增加了 Q&amp;A 我將 Q&amp;A 拆成了兩個 part ,一個是給要深入專案的,一個是給第一次參與的

👀 2
陳昱廷 14:42:16
大家可以討論一下
陳昱廷 16:29:39
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> • #0 *:pick: 變更內容 / Details of Changes* 增加 smtp provider

陳昱廷 16:29:48
這個 pr 也有幾個地方要討論的
陳昱廷 16:29:52
大家可以討論一下
CodingMan 18:36:06
@thetruth0906 has joined the channel
🙌 1
nickyanggg 21:44:19
再麻煩幫忙 review,感謝🙏
https://github.com/Ptt-official-app/go-bbs/pull/80

#80 Feature/#79 看板發文限制以及細部資訊

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/go-bbs/issues/79|#79> *:pick: 變更內容 / Details of Changes* • 在 pttbbs/board.go 中新增 InfoAndSettings • 在 pttbbs/board_test.go 中新增 TestInfoAndSettings

👍 1

2021-04-30

pichuchen 00:28:35
https://github.com/Ptt-official-app/go-bbs/issues/82
go-bbs 開了一個 ISSUE

#82 [主線] [PTT] 實作在文章底部新增一行文字的method

*實作細節 / Details of Implement* • 目前在 <https://github.com/Ptt-official-app/Ptt-backend/issues/194|實作文章推文> 以及 <https://github.com/Ptt-official-app/Ptt-backend/issues/66|轉錄看板文章> 都需要直接新增一行字串 • 直接編輯整個文章和在後面新增一行字串在硬碟上的效率不太一樣,新增字串會有比較高的效能 • 可能會在原有文章或是 bbs 的 connecter 新增一個 method 裡面有參數來表示某個看板的某個文章,然後和一行任意文字。 *期程 / Schedule* (希望這個任務大概什麼時候做完,如果會卡到其他地方的功能也請在這邊提出。) • 討論時間: 三天,5/2 • 實作時間: 一週,5/9 • 確認時間: 一週,5/16 *相關文件 / Documents* 基本上是為了實作這樣的功能 <https://user-images.githubusercontent.com/600238/116584922-71fcd080-a94a-11eb-9e6b-7eb0c068e71b.png|截圖 2021-04-30 上午12 24 12>

👀 1
titaneric 13:20:19
以下PR需要大家幫忙review,感謝 https://github.com/Ptt-official-app/Ptt-backend/pull/202

#202 推文repo更新

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/194|#194> *:pick: 變更內容 / Details of Changes* • `AppendComment`統一回傳`PushRecord` interface,這需要在go-bbs更新 • 改寫`AppendComment` response body in delivery,使其與文件一致 • 測試`AppendComment` repo mocked method

👀 1